Skip to content

Conversation

@NirajC-Microsoft
Copy link
Contributor

@NirajC-Microsoft NirajC-Microsoft commented Oct 7, 2025

Purpose

This pull request introduces several improvements to the post-deployment experience for the Document Generation Solution Accelerator, focusing on better guidance, automation, and resource tagging. The main changes include the addition of a comprehensive post-deployment guide, enhancements to the sample data import script for improved parameter handling and Azure authentication, and updates to the infrastructure code for more informative outputs and resource tagging.

Documentation improvements:

  • Added a new AVMPostDeploymentGuide.md that provides step-by-step instructions for post-deployment tasks, including importing sample data, configuring authentication, and cleaning up resources after failed deployments.

Sample data import automation:

  • Refactored process_sample_data.sh to automatically retrieve deployment parameters from Azure resource group tags and deployment outputs, with fallback to manual input if the deployment does not exist. Also added Azure CLI authentication logic for smoother script execution. [1] [2]

Infrastructure outputs and tagging:

  • Updated main.bicep to output additional deployment metadata, including the subscription ID and managed identity client ID, which improves traceability and supports downstream automation.
  • Added a DeploymentName tag to the resource group for easier identification and correlation of resources with deployments.
  • ...

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

  • I have built and tested the code locally and in a deployed app
  • For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
  • This is a change for all users of this app. No code or asset is specific to my use case or my organization.

Other Information

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