Skip to content

Conversation

NirajC-Microsoft
Copy link
Contributor

Purpose

This pull request introduces improvements to the deployment and post-deployment process for the Azure infrastructure, focusing on enhanced guidance, script usability, and resource output clarity. The main changes include updating deployment scripts and documentation to consistently require the resource group name as an argument, expanding the sample data processing script to retrieve additional managed identity and database information, and exposing the Azure subscription ID as an output in the Bicep template.

Deployment and Script Usability Improvements

  • Updated deployment instructions and output messages in both PowerShell and Bash hooks in azure.yaml to explicitly include the resource group name as an argument when running process_sample_data.sh, improving clarity and reducing user error. [1] [2]
  • Revised the post-deployment documentation in docs/AVMPostDeploymentGuide.md to match the updated script invocation syntax, ensuring users run the script correctly.

Infrastructure Output Enhancements

  • Added an output for the Azure subscription ID in infra/main.bicep, making it available for downstream scripts and documentation.

Sample Data Script Expansion

  • Enhanced infra/scripts/process_sample_data.sh to automatically retrieve and display additional deployment outputs, including managed identity client IDs and display names for both the web app and SQL database, as well as the SQL database name.
  • Updated the manual input section in the same script to prompt for SQL managed identity details, ensuring all required information is collected when the deployment outputs are unavailable.
  • ...

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

@Prajwal-Microsoft Prajwal-Microsoft merged commit 21e8b9a into dev Oct 9, 2025
14 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.

2 participants