Skip to content

feat: Enhance deployment pipeline, modularize test workflows, simplify infra, and update docs #140

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 58 commits into from
Jun 20, 2025

Conversation

Shreyas-Microsoft
Copy link
Collaborator

Purpose

This pull request introduces significant updates to the deployment pipeline, test automation workflows, infrastructure configuration, and documentation. The changes aim to enhance deployment flexibility, improve test automation integration, streamline infrastructure definitions, and clarify documentation for end-users.

Deployment Pipeline Enhancements:

  • Updated .github/workflows/deploy.yml to trigger on workflow_run and added support for multiple branches (main, dev, demo) and manual dispatch. Introduced new environment variables (GPT_MIN_CAPACITY, BRANCH_NAME) to customize deployments.
  • Added logic to dynamically set the image tag based on the branch name and updated Bicep deployment parameters to include capacity and imageVersion.
  • Introduced e2e-test and cleanup-deployment jobs to automate testing and resource cleanup post-deployment.

Test Automation Workflow Updates:

  • Refactored .github/workflows/test-automation.yml to use workflow_call for modular integration, allowing inputs like CODEMOD_WEB_URL. Removed container app start/stop steps to simplify the workflow. [1] [2] [3]
  • Added a step to open the provided URL for testing purposes.

Infrastructure and Configuration Changes:

  • Updated infra/deploy_ai_foundry.bicep to remove unused resources like storage accounts and AI Hub, simplifying the Bicep template.
  • Added new abbreviations for AI Foundry in infra/abbreviations.json.

Documentation Improvements:

  • Updated default Azure locations and clarified parameters in docs/CustomizingAzdParameters.md and docs/DeploymentGuide.md. Added a new parameter AZURE_AISERVICE_LOCATION for AI service location. [1] [2]

Minor Fixes:

  • Adjusted scripts in azure.yaml to ensure proper handling of parameters and enabled interactive mode for better user experience.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

Priyanka-Microsoft and others added 30 commits June 10, 2025 18:26
fix: prefer user region then all other region
…-check

feat: implement interactive quota check script with user inputs
@Roopan-Microsoft Roopan-Microsoft merged commit b5d5989 into main Jun 20, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants