Skip to content

fix: downmerge from main to dev#473

Merged
Roopan-Microsoft merged 12 commits into
devfrom
main
Jul 18, 2026
Merged

fix: downmerge from main to dev#473
Roopan-Microsoft merged 12 commits into
devfrom
main

Conversation

@VishalSh-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request updates the project to use the latest GPT-5.1 model and introduces a dedicated Azure Container Registry (ACR) for each deployment, along with improvements to documentation and deployment scripts. The most important changes are grouped below:

GPT Model Upgrade and Configuration

  • Default GPT model has been changed from gpt-4o to gpt-5.1 throughout infrastructure templates (main.bicep, main_custom.bicep), parameter files, and documentation. The default model version is now 2025-11-13. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]
  • Documentation and quota check scripts updated to reference gpt-5.1 as the required/default model, including usage examples and quota recommendations. [1] [2] [3] [4]

Container Registry and Image Build Process

  • Dedicated Azure Container Registry is now provisioned per deployment via new module in main.bicep, with private networking support and required DNS configuration. [1] [2] [3]
  • Container apps are initially deployed with a public placeholder image; a new documented post-provision step builds and pushes backend/frontend images to the dedicated ACR, then updates the apps. [1] [2] [3] [4]
  • Outputs for the ACR endpoint, name, and image tag are exposed for use by deployment scripts.
  • Parameters related to a static container registry endpoint have been removed from parameter files, reflecting the new per-deployment registry. [1] [2]

Deployment and Documentation Improvements

  • Deployment guide updated to include new build and push image steps, and step numbering adjusted to reflect the new workflow. [1] [2]
  • Environment variable documentation clarified for new model defaults and container image handling.

These changes ensure the project uses the latest GPT model, improves security and isolation by providing a dedicated container registry per deployment, and offers clearer deployment instructions.

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.

VishalSh-Microsoft and others added 5 commits July 14, 2026 12:42
chore: Update Owners ID for Vinay and Prajwal
…-vs7

feat: add Azure Container Registry module, enhance deployment scripts and update model configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the accelerator’s default GPT model configuration to gpt-5.1 (version 2025-11-13) and changes the container image flow to provision a dedicated Azure Container Registry (ACR) per deployment, with post-provision scripts to build/push images and update Container Apps accordingly.

Changes:

  • Upgrade default GPT model references across infra, scripts, samples, and documentation (gpt-4ogpt-5.1, version → 2025-11-13).
  • Introduce per-deployment ACR provisioning (new Bicep module) and switch Container Apps to start from a public placeholder image, then update to ACR images via new build/push scripts.
  • Remove static container registry endpoint parameters from infra parameter files and update deployment/quota documentation.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/backend/.env.sample Updates sample agent model deployment names to gpt-5.1.
scripts/quota_check_params.sh Updates default quota-check model/capacity to gpt-5.1:150.
scripts/checkquota.sh Updates quota check model key to OpenAI.GlobalStandard.gpt-5.1.
scripts/build_and_push_images.sh Adds Bash post-provision script to build/push images to the per-deployment ACR and update Container Apps.
scripts/build_and_push_images.ps1 Adds PowerShell post-provision script to build/push images to the per-deployment ACR and update Container Apps.
infra/modules/containerRegistry.bicep Adds a reusable module to deploy ACR (+ optional private endpoint + role assignments).
infra/main.waf.parameters.json Removes the now-obsolete containerRegistryEndpoint parameter.
infra/main.parameters.json Removes the now-obsolete containerRegistryEndpoint parameter.
infra/main.json Regenerates ARM JSON reflecting ACR module, placeholder images, and new outputs.
infra/main.bicep Provisions a dedicated ACR per deployment, configures Container Apps registry identity, uses a placeholder image, and exports ACR/app outputs for scripts.
infra/main_custom.bicep Updates default GPT model/version in the custom infra template.
docs/quota_check.md Updates documentation examples to gpt-5.1.
docs/EXAMPLE-CustomizingAzdParameters.md Updates model customization example to AZURE_ENV_GPT_MODEL_NAME gpt-5.1.
docs/DeploymentGuide.md Adds a new “Build and Push Container Images” step and adjusts step numbering/duration.
docs/CustomizingAzdParameters.md Updates default GPT model/version values and removes container registry endpoint parameter documentation.
docs/AzureGPTQuotaSettings.md Updates quota guidance to reference GPT-5.1.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/build_and_push_images.ps1
Comment thread infra/modules/containerRegistry.bicep
Comment thread docs/DeploymentGuide.md Outdated
Comment thread docs/DeploymentGuide.md Outdated
Copilot AI review requested due to automatic review settings July 16, 2026 11:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.

Comment thread infra/main.bicep
Comment thread scripts/build_and_push_images.sh

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Comment thread scripts/build_and_push_images.sh
Comment thread infra/main.bicep
@Roopan-Microsoft
Roopan-Microsoft merged commit 0b2edd8 into dev Jul 18, 2026
15 of 22 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.

6 participants