Skip to content

Conversation

Kanchan-Microsoft
Copy link
Contributor

Purpose

This pull request introduces several improvements and additions to deployment documentation, error troubleshooting, and Azure resource tagging. The most significant changes include expanded troubleshooting steps for Azure deployments, new guides for building and pushing container images, and enhancements to resource tagging and deployment metadata for improved traceability and compliance.

Documentation Improvements

  • Added a comprehensive guide docs/ACRBuildAndPushGuide.md for building and pushing Docker images to Azure Container Registry, including step-by-step instructions for both WebApp and Backend services.
  • Introduced docs/DeployWithLimitedQuota.md to help users deploy with limited Azure OpenAI quota, including instructions for bypassing or lowering quota validation in the Bicep template.

Troubleshooting Enhancements

  • Expanded docs/TroubleShootingSteps.md with many new error scenarios, resolutions, and links to official Microsoft documentation, including errors like InvalidRequestContent, ReadOnlyDisabledSubscription, SkuNotAvailable, CrossTenantDeploymentNotPermitted, SpecialFeatureOrQuotaIdRequired, and more.
  • Updated error descriptions and solutions for resource ID formatting and resource provider registration, improving clarity and accuracy.

Deployment & Tagging Improvements

  • Enhanced resource group tagging in infra/main.bicep by adding a Type tag to distinguish between WAF and Non-WAF deployments, and improved logic for the CreatedBy tag to use either the username or object ID.
  • In .github/workflows/CAdeploy.yml, added automatic timestamp tagging for deployments, improving auditability and resource lifecycle management.

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

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.

1 participant