feat: merging the changes from dev to main#120
Merged
Merged
Conversation
fix: fixes for the Bug Bash feedback
feat: added opentelemetry logs
feat: Standardize Bicep Parameters for Code Modernization
feat: Added Quota Auto Validation Scripts
Abdul-Microsoft
requested review from
Avijit-Microsoft,
Prajwal-Microsoft,
Roopan-Microsoft,
Vinay-Microsoft,
aniaroramsft and
marktayl1
as code owners
June 5, 2025 14:45
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request merges development changes into main that enhance our development environment, CI/CD workflows, and Azure infrastructure while updating documentation and parameter handling.
- Development Environment: Added Azure CLI/Bicep support in devcontainer and updated VS Code extensions.
- CI/CD Workflow: Introduced a new GitHub Actions workflow for test automation with retries and notifications.
- Azure Infrastructure & Documentation: Updated Bicep templates, parameter definitions, and documentation to improve deployment flexibility and clarity.
Reviewed Changes
Copilot reviewed 35 out of 35 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/e2e-test/.gitignore | Added common ignore rules for Python and project-related files. |
| src/backend/requirements.txt | Added OpenTelemetry and Azure monitor dependencies (with one duplicate). |
| src/backend/api/event_utils.py | Introduced Application Insights event tracking helper function. |
| src/backend/api/api_routes.py | Integrated telemetry tracking and error reporting into API routes. |
| scripts/validate_model_quota.sh | Added quota validation logic for model deployment using Azure CLI. |
| scripts/validate_model_quota.ps1 | PowerShell version of model quota validation. |
| scripts/validate_model_deployment_quota.sh | Added shell script for validating model deployment quotas. |
| scripts/validate_model_deployment_quota.ps1 | PowerShell variant for validating deployment quotas. |
| scripts/quota_check_params.sh | Updated default regions and parameter checks for quota query. |
| infra/main.parameters.json | Added AI model deployment parameters. |
| infra/main.json | Updated infrastructure JSON with parameterized values. |
| infra/main.bicepparam | Enhanced Bicep parameter file with new environment parameters. |
| infra/main.bicep | Updated Bicep template to use new parameters and location settings. |
| infra/deploy_ai_foundry.bicep | Updated AI Foundry deployment to include Application Insights. |
| docs/EXAMPLE-CustomizingAzdParameters.md | Fixed typos and clarified parameter instructions. |
| docs/CustomizingAzdParameters.md | Expanded parameter table and improved instructions. |
| azure.yaml | Renamed parameters and added preprovision hooks for quota validation. |
| .github/workflows/test-automation.yml | Introduced workflow for test automation with scheduled and manual triggers. |
| .devcontainer/devcontainer.json | Updated devcontainer configuration with new features and post-commands. |
Comments suppressed due to low confidence (2)
src/backend/api/api_routes.py:1053
- An HTTPException is returned instead of being raised. Replace 'return HTTPException(...)' with 'raise HTTPException(...)' to properly signal the error.
return HTTPException(status_code=404, detail="No batch history found.")
azure.yaml:28
- The same chmod command is executed twice; consider removing the duplicate to simplify the preprovision hook commands.
chmod u+r+x ./scripts/validate_model_deployment_quota.sh; chmod u+r+x ./scripts/validate_model_deployment_quota.sh;
Roopan-Microsoft
approved these changes
Jun 5, 2025
Avijit-Microsoft
approved these changes
Jun 5, 2025
|
🎉 This PR is included in version 1.4.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Ritesh-Microsoft
pushed a commit
that referenced
this pull request
Oct 10, 2025
feat: merging the changes from dev to main
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Purpose
This pull request includes significant updates to the development environment, CI/CD workflows, Azure infrastructure, and documentation. These changes enhance the automation, flexibility, and maintainability of the project. Below is a summary of the most important changes grouped by theme.
Development Environment Enhancements
.devcontainer/devcontainer.jsonfile, enabling easier management of Azure resources. Also added new VS Code extensions (ms-azuretools.vscode-azcli) and a post-create command to set permissions for a script.CI/CD Workflow Additions
.github/workflows/test-automation.yml) for test automation. This workflow includes scheduled runs, retries for flaky tests, and notifications for success or failure. It also starts/stops Azure Container Apps and uploads test reports.Azure Infrastructure Updates
infra/main.bicepand related files to improve flexibility:Documentation Improvements
docs/CustomizingAzdParameters.md) by adding a detailed parameters table and examples for customizing deployment configurations.Azure YAML Configuration Updates
azure.yamlfor consistency, and added pre-provision hooks to validate quotas for AI model deployments.Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information