feat: Refactor infrastructure parameters and add image build scripts#467
Merged
Conversation
…d_and_push_images.sh) for building and pushing container images to ACR.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the deployment workflow to provision a dedicated Azure Container Registry (ACR) per deployment and (intendedly) automate building/pushing the backend/frontend container images via remote ACR builds, while also cleaning up infrastructure parameters and refreshing default GPT model settings.
Changes:
- Added post-provision scripts to remotely build backend/frontend images with
az acr buildand update Container Apps to the newly built images. - Updated infrastructure to provision a per-deployment ACR, configure Container Apps with a placeholder image, and output ACR/app metadata for scripts.
- Removed the now-unneeded
containerRegistryEndpointparameter from parameter files and updated docs to reflect the new workflow (plus updated default GPT model name/version inmain.bicep).
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/build_and_push_images.sh | New Bash script to build/push images in ACR and update Container Apps. |
| scripts/build_and_push_images.ps1 | New PowerShell script to build/push images in ACR and update Container Apps. |
| infra/main.waf.parameters.json | Removes containerRegistryEndpoint parameter value wiring. |
| infra/main.parameters.json | Removes containerRegistryEndpoint parameter value wiring. |
| infra/main.json | Regenerated ARM template reflecting ACR + placeholder image + outputs. |
| infra/main.bicep | Provisions ACR per deployment, sets placeholder images, adds outputs for post-provision image build/update. |
| infra/main_custom.bicep | Updates default GPT model name (but currently not the version). |
| docs/DeploymentGuide.md | Documents the new image build process and Azure CLI requirements. |
| docs/CustomizingAzdParameters.md | Updates AZURE_ENV_IMAGE_TAG description and removes container registry endpoint env var. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Roopan-Microsoft
approved these changes
Jul 3, 2026
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 introduces a major improvement to the container image build and deployment process by provisioning a dedicated Azure Container Registry (ACR) for every deployment and automating backend/frontend image builds using remote ACR builds. The infrastructure and documentation have been updated to reflect this new workflow, and new post-provision scripts have been added to handle the image build and deployment process. There are also updates to default GPT model versions and several infrastructure parameter cleanups.
Container Image Build & Deployment Automation
scripts/build_and_push_images.shandscripts/build_and_push_images.ps1) that remotely build backend and frontend images inside the deployment's ACR usingaz acr buildand update the container apps to use the freshly built images. These scripts ensure the Azure CLI is authenticated and handle all required environment values. [1] [2]infra/main.bicepto provision a dedicated Azure Container Registry per deployment, output relevant ACR and container app information, and configure container apps to use a placeholder image initially, which is replaced by the post-provision script. [1] [2] [3] [4] [5]Infrastructure Parameter and Template Updates
imageTagand related descriptions to clarify that tags are applied to images built and pushed remotely, and removed references to manual registry endpoints in documentation and infrastructure. [1] [2]Documentation Improvements
Model Version Updates
infra/main.bicepandinfra/main_custom.biceptogpt-5.1and2025-11-13, respectively. [1] [2]These changes collectively streamline the deployment workflow, reduce manual configuration, and improve reliability and clarity for users and contributors.
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation