-
Notifications
You must be signed in to change notification settings - Fork 181
chore: Dev to Main merge #803
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
96 commits
Select commit
Hold shift + click to select a range
86d2279
refactor(frontend): complete Tasks 1-5 UI refactoring
Shubhangi-Microsoft 653846d
refactor(frontend): Task 6 — Modularize utility functions by domain
Shubhangi-Microsoft 8007ba0
refactor(frontend): remove dead code, deduplicate error handling, tig…
Shubhangi-Microsoft 50bac33
refactor(frontend): dedup streamChat, remove dead code, tighten impor…
Shubhangi-Microsoft 3216286
refactor(frontend): GenerationStatus enum, delete dead code, dedup va…
Shubhangi-Microsoft bfce2ef
refactor: clean up unused barrel exports and standardize httpClient i…
Shubhangi-Microsoft f336f3b
refactor: extract AI_DISCLAIMER constant, remove dead selector and tr…
Shubhangi-Microsoft ddf6a0c
chore: remove 3 unused barrel exports (setShowChatHistory, parseTextC…
Shubhangi-Microsoft 4f22f79
refactor: eliminate remaining Task 2/6/8 issues
Shubhangi-Microsoft 8a7d3b4
refactor: extract readSSEResponse helper, un-export GENERATION_STATUS…
Shubhangi-Microsoft 1650f11
fix: wrap MessageBubble copy handler in useCallback
Shubhangi-Microsoft 3e54b33
merge: resolve conflicts with dev (keep refactored App.tsx and api/in…
Shubhangi-Microsoft 0f57d1e
merge: resolve conflicts with restructured dev branch
Shubhangi-Microsoft 10ede7e
fix: resolve 6 Copilot review comments on PR #768
Shubhangi-Microsoft 1252f6f
docs: add VM size recommendations for Bastion and Jumpbox subnets in …
Yatish-Microsoft 7b83c0c
Add Bastion and Jumpbox VM using existing VNet configuration.
Yatish-Microsoft 731c130
Merge branch 'dev' into dev-yat
Yatish-Microsoft 816f8c9
updated main.json
Yatish-Microsoft 4613504
Refactor environment variable names for Azure OpenAI settings
Pavan-Microsoft 4211d9d
fix: rename environment variable for container registry in Docker bui…
Pavan-Microsoft 947ae2d
refactor: rename image_model_choice to AZURE_ENV_IMAGE_MODEL_NAME acr…
Pavan-Microsoft bd0aff5
fix: update environment variable references for image model in docume…
Pavan-Microsoft 14b3547
fix: update usage instructions and add legacy environment variable su…
Pavan-Microsoft 45f9efa
Merge remote-tracking branch 'origin/dev' into dev-yat
Yatish-Microsoft 8cf7f41
feat: add support for deploying Azure Bastion and Jumpbox resources i…
Yatish-Microsoft 887db52
fix: resolve Copilot review comments - deployment condition, output c…
Yatish-Microsoft 36d3815
fix: honor deployBastionAndJumpbox independently from VM password
Yatish-Microsoft b518ac8
fix: address remaining PR 777 copilot comments
Yatish-Microsoft ef25e8a
fix: add RG_TAGS environment variable for resource group creation
Vamshi-Microsoft 2d6e64a
fix: align VNet subnet deployment with deployAdminAccessResources con…
Copilot 7d89978
Merge remote-tracking branch 'origin/dev' into psl-pk-standparam
Pavan-Microsoft aaf49e7
commit
Tejasri-Microsoft 7abd216
commit
Tejasri-Microsoft d527f13
commit
Tejasri-Microsoft 4e5e944
commit
Tejasri-Microsoft fab923f
commit
Tejasri-Microsoft b193a5c
commit
Tejasri-Microsoft 14f61f8
refactor: restructure frontend folder - move contents to app/, rename…
Shubhangi-Microsoft 3d05925
fix: address PR review - use npm ci --omit=dev, create server/static …
Shubhangi-Microsoft 7494e38
chore: remove event_utils changes from PR
Yatish-Microsoft dafa23d
fix: rename param name to vnetName to avoid collision with output nam…
Copilot bec9fdb
commit
Tejasri-Microsoft 5f6e0dd
commit
Tejasri-Microsoft 2a10da3
fix: correct azd-template-validation workflow indentation
Tejasri-Microsoft 919d16d
commit
Tejasri-Microsoft dcfd389
Filter the paths for pipeline run
Prekshith-Microsoft a5d6a5c
merge: integrate psl-uitemplatechng into psl-ui-refractoring (folder …
Shubhangi-Microsoft 1f3ea2f
add Bicep parameter validation workflow and script
Harsh-Microsoft af14382
update workflow triggers and notification conditions in validate-bice…
Harsh-Microsoft 89b6822
include validate_bicep_params.py in workflow paths
Harsh-Microsoft 939b9eb
Changes for deployment of custom changes
Ragini-Microsoft 93c82af
updates for custom deployment
Ragini-Microsoft 090821c
add deployment instructions for local changes
Ragini-Microsoft c07d99c
fix: update link to local development guide in documentation
Ragini-Microsoft 8a74f59
feat: Add VM size recommendations for Bastion and Jumpbox subnets
Avijit-Microsoft eff8b3d
Merge pull request #794 from microsoft/psl-content-gen-templatev
Prajwal-Microsoft 1409848
Fixed the code quality issue
Prekshith-Microsoft 5dccce1
Merge pull request #802 from microsoft/psl-codeql
Roopan-Microsoft 9c51956
updates in custom deployment
Ragini-Microsoft 62eba03
Merge branch 'dev' of https://github.com/microsoft/content-generation…
Ragini-Microsoft 1cb6af4
update custom file as per main.bicep changes
Ragini-Microsoft 7b04179
chore: Add support for deploying local changes via azd up
Avijit-Microsoft b9b95ef
refactor: rename src/app to src/App and update all path references
Shubhangi-Microsoft fd4f1ab
Merge remote-tracking branch 'origin/dev' into psl-pk-standparam
Pavan-Microsoft f6722be
fix: address PR review - ChatInput event type, userId fallback, conte…
Shubhangi-Microsoft 06c7787
Revert "fix: address PR review - ChatInput event type, userId fallbac…
Shubhangi-Microsoft a812b18
Refactor Azure resource identifiers in workflows and scripts
Pavan-Microsoft 462a53d
fix: Simplify first run check for ACI deployment in azure_custom.yaml
Ragini-Microsoft 1e74f69
refactor: update Azure environment variable names and remove telemetr…
Pavan-Microsoft 8bf12c6
fix: simplify check for ACI deployment in custom deployment
Avijit-Microsoft bd46670
Merge pull request #805 from microsoft/main
Avijit-Microsoft e74493f
fix: add bicep version requirement (>= 0.33.0) to azure.yaml
Roopan-Microsoft 9bccc87
fix: Remove create-release.yml path filter changes
Prekshith-Microsoft 9f272f9
refactor: update environment variable names to use AZURE_ENV prefix
Pavan-Microsoft 8d1866b
Merge remote-tracking branch 'origin/dev' into psl-pk-standparam
Pavan-Microsoft 3dcd660
fix: address Copilot review comments - deploy.sh Dockerfile path and …
Pavan-Microsoft eff00ed
fix: uncomment metadata section in azure.yaml
Pavan-Microsoft c5b4086
fix: add bicep version requirement (>= 0.33.0) to azure.yaml
Avijit-Microsoft c3cd3ed
fix: deploy.ps1 Dockerfile path and remove eastus from checkquota def…
Pavan-Microsoft c72e70e
fix: update environment variable names to use AZURE_ prefix in deploy…
Pavan-Microsoft b028039
Merge pull request #787 from microsoft/psl-pk-standparam
Roopan-Microsoft 9fb8bc0
Merge pull request #799 from microsoft/hb-psl-38859
Roopan-Microsoft d4bd501
fix: improve error handling for JSON parsing
Ragini-Microsoft db4409e
fix: improve error handling for JSON parsing
Avijit-Microsoft 7f3ec7d
Update README
Thanusree-Microsoft ed4ee46
Update link in README for deployment guide
Thanusree-Microsoft f325d27
Fix link to deployment configuration in README
Thanusree-Microsoft d099068
Update deployment documentation with security notes
Thanusree-Microsoft 8ee3dca
Fix link to AZD_DEPLOYMENT.md in deployment notes
Thanusree-Microsoft c77a108
Merge pull request #768 from microsoft/psl-ui-refractoring
Roopan-Microsoft 0a42e28
Merge pull request #808 from microsoft/psl-sfiissue
Roopan-Microsoft 34d01a3
fix: Resolve merge conflicts with dev branch
Prekshith-Microsoft 45d9c08
fix: Restore push trigger with path filters for azure-dev.yml
Prekshith-Microsoft 2fafeb4
Merge pull request #798 from microsoft/psl-filter-paths
Roopan-Microsoft 9af260a
Revert PR #768: UI refactoring
Pavan-Microsoft 39dab72
Merge pull request #810 from microsoft/revert-pr-768
Roopan-Microsoft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| name: AZD Template Validation | ||
| on: | ||
| schedule: | ||
| - cron: '30 1 * * 4' # Every Thursday at 7:00 AM IST (1:30 AM UTC) | ||
| workflow_dispatch: | ||
|
|
||
| permissions: | ||
| contents: read | ||
| id-token: write | ||
| pull-requests: write | ||
| jobs: | ||
| template_validation: | ||
| runs-on: ubuntu-latest | ||
| environment: production | ||
| name: azd template validation | ||
| env: | ||
| GH_TOKEN: ${{ github.token }} | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Set timestamp | ||
| shell: bash | ||
| run: echo "HHMM=$(date -u +'%H%M')" >> "$GITHUB_ENV" | ||
|
|
||
| - name: Validate Azure Template | ||
| uses: microsoft/template-validation-action@v0.4.3 | ||
| with: | ||
| validateAzd: ${{ vars.TEMPLATE_VALIDATE_AZD }} | ||
| validateTests: ${{ vars.TEMPLATE_VALIDATE_TESTS }} | ||
| useDevContainer: ${{ vars.TEMPLATE_USE_DEV_CONTAINER }} | ||
| id: validation | ||
| env: | ||
| AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }} | ||
| AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }} | ||
| AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }} | ||
| AZURE_ENV_NAME: ${{ secrets.AZURE_ENV_NAME }}-${{ env.HHMM }} | ||
| AZURE_LOCATION: ${{ secrets.AZURE_LOCATION }} | ||
| AZURE_ENV_AI_SERVICE_LOCATION: ${{ secrets.AZURE_ENV_AI_SERVICE_LOCATION }} | ||
| AZURE_AI_MODEL_CAPACITY: 1 | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: print result | ||
| run: cat ${{ steps.validation.outputs.resultFile }} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,52 +1,80 @@ | ||
| name: Azure Template Validation | ||
| name: Azure Dev Deploy | ||
| on: | ||
| workflow_dispatch: | ||
| push: | ||
| branches: | ||
| - main | ||
| paths: | ||
| - 'infra/**' | ||
| - 'azure*.yaml' | ||
| - '.github/workflows/azure-dev.yml' | ||
|
|
||
|
|
||
| permissions: | ||
| contents: read | ||
| id-token: write | ||
| pull-requests: write | ||
|
|
||
| jobs: | ||
| template_validation_job: | ||
| deploy: | ||
| runs-on: ubuntu-latest | ||
| name: azd deploy | ||
| environment: production | ||
| name: Template validation | ||
|
|
||
| env: | ||
| AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }} | ||
| AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }} | ||
| AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }} | ||
| AZURE_ENV_NAME: ${{ secrets.AZURE_ENV_NAME }} | ||
| AZURE_LOCATION: ${{ secrets.AZURE_LOCATION }} | ||
| AZURE_ENV_AI_SERVICE_LOCATION: ${{ secrets.AZURE_ENV_AI_SERVICE_LOCATION }} | ||
| AZURE_ENV_EXISTING_LOG_ANALYTICS_WORKSPACE_RID: ${{ secrets.AZURE_ENV_EXISTING_LOG_ANALYTICS_WORKSPACE_RID }} | ||
| AZURE_EXISTING_AIPROJECT_RESOURCE_ID: ${{ secrets.AZURE_EXISTING_AIPROJECT_RESOURCE_ID }} | ||
| AZURE_DEV_COLLECT_TELEMETRY: ${{ vars.AZURE_DEV_COLLECT_TELEMETRY }} | ||
|
|
||
| steps: | ||
| # Step 1: Checkout the code from your repository | ||
| - name: Checkout code | ||
| - name: Checkout Code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| # Step 2: Pre-authenticate Azure for azd validation | ||
| - name: Login to Azure | ||
| - name: Set timestamp and env name | ||
| shell: bash | ||
| run: | | ||
| az login --service-principal -u "${{ secrets.AZURE_CLIENT_ID }}" -p "${{ secrets.AZURE_CLIENT_SECRET }}" --tenant "${{ secrets.AZURE_TENANT_ID }}" | ||
| az account set --subscription "${{ secrets.AZURE_SUBSCRIPTION_ID }}" | ||
|
|
||
| # Step 3: Validate the Azure template using microsoft/template-validation-action | ||
| - name: Validate Azure Template | ||
| uses: microsoft/template-validation-action@v0.4.3 | ||
| HHMM=$(date -u +'%H%M') | ||
| echo "AZURE_ENV_NAME=${AZURE_ENV_NAME}-${HHMM}" >> "$GITHUB_ENV" | ||
|
|
||
| - name: Install azd | ||
| uses: Azure/setup-azd@v2 | ||
|
|
||
| - name: Login to Azure | ||
| uses: azure/login@v2 | ||
| with: | ||
| workingDirectory: . | ||
| validateAzd: ${{ vars.TEMPLATE_VALIDATE_AZD }} | ||
| useDevContainer: ${{ vars.TEMPLATE_USE_DEV_CONTAINER }} | ||
| validateTests: ${{ vars.TEMPLATE_VALIDATE_TESTS }} | ||
| id: validation | ||
| env: | ||
| AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }} | ||
| AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }} | ||
| AZURE_CLIENT_SECRET: ${{ secrets.AZURE_CLIENT_SECRET }} | ||
| AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }} | ||
| AZURE_ENV_NAME: ${{ secrets.AZURE_ENV_NAME }} | ||
| AZURE_LOCATION: ${{ secrets.AZURE_LOCATION }} | ||
| AZURE_ENV_OPENAI_LOCATION: ${{ secrets.AZURE_ENV_OPENAI_LOCATION }} | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| # Step 4: Print the result of the validation | ||
| - name: Print result | ||
| run: cat ${{ steps.validation.outputs.resultFile }} | ||
| client-id: ${{ secrets.AZURE_CLIENT_ID }} | ||
| tenant-id: ${{ secrets.AZURE_TENANT_ID }} | ||
| subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }} | ||
|
|
||
| - name: Login to AZD | ||
| shell: bash | ||
| run: | | ||
| azd auth login \ | ||
| --client-id "$AZURE_CLIENT_ID" \ | ||
| --federated-credential-provider "github" \ | ||
| --tenant-id "$AZURE_TENANT_ID" | ||
|
|
||
| - name: Provision and Deploy | ||
| shell: bash | ||
| run: | | ||
| if ! azd env select "$AZURE_ENV_NAME"; then | ||
| azd env new "$AZURE_ENV_NAME" --subscription "$AZURE_SUBSCRIPTION_ID" --location "$AZURE_LOCATION" --no-prompt | ||
| fi | ||
|
|
||
| azd config set defaults.subscription "$AZURE_SUBSCRIPTION_ID" | ||
|
|
||
|
|
||
| if [[ -n "${AZURE_ENV_EXISTING_LOG_ANALYTICS_WORKSPACE_RID:-}" ]]; then | ||
| azd env set AZURE_ENV_EXISTING_LOG_ANALYTICS_WORKSPACE_RID "$AZURE_ENV_EXISTING_LOG_ANALYTICS_WORKSPACE_RID" | ||
| fi | ||
|
|
||
| if [[ -n "${AZURE_EXISTING_AIPROJECT_RESOURCE_ID:-}" ]]; then | ||
| azd env set AZURE_EXISTING_AIPROJECT_RESOURCE_ID "$AZURE_EXISTING_AIPROJECT_RESOURCE_ID" | ||
| fi | ||
|
|
||
| azd up --no-prompt | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.