-
Notifications
You must be signed in to change notification settings - Fork 50
Simplify infrastructure deployment with Bicep parameters and rename global resource group #793
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
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
f5d7567 to
6b6023f
Compare
Approve Database Migration
|
a587ee5 to
8c4c743
Compare
… instead of using batch
…stic when SQL Server is not fully provisioned
…Auto' bindingType for single-deployment custom domain
8c4c743 to
81d8753
Compare
…me to eliminate what-if warnings
… global resource groups
…que revisions per deployment
81d8753 to
6e2476e
Compare
|
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
Merged
2 tasks
tjementum
added a commit
that referenced
this pull request
Dec 11, 2025
…taging registry reference (#803) ### Summary & Motivation Fix production container deployments failing to import images from the staging registry. The `STAGING_REGISTRY_ID` in `_deploy-container.yml` was missing the `-global` suffix in the resource group name, a bug introduced in #793 when the global resource group was renamed. This is currently preventing all deployments to production. - Update resource group reference from `{prefix}-stage` to `{prefix}-stage-global` to match the actual location of the staging container registry ### Checklist - [x] I have added tests, or done manual regression tests - [x] I have updated the documentation, if necessary
Merged
2 tasks
tjementum
added a commit
that referenced
this pull request
Dec 16, 2025
… in Azure Container Apps (#804) ### Summary & Motivation Upgrade the Azure Container Apps API version from `2025-07-01` (stable) to `2025-10-02-preview` to restore the `autoConfigureDataProtection` runtime setting for .NET containers. When upgrading to the stable `2025-07-01` API version in PR #793, the `autoConfigureDataProtection` feature was inadvertently lost because it remains a preview-only feature not included in stable releases. This change reverts to a preview API version that supports this feature, which enables automatic configuration of ASP.NET Data Protection keys across all container apps in an environment - essential for antiforgery token validation when multiple container apps (AppGateway, account-management, back-office, etc.) need to share encrypted tokens. The learning here is that stable API versions may not include all preview features that the infrastructure depends on. When upgrading API versions, verify that preview features are still available or consciously choose to remain on a preview version until the required features become stable. ### Checklist - [x] I have added tests, or done manual regression tests - [x] I have updated the documentation, if necessary
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.
Summary & Motivation
Replace inline Bicep parameter strings with structured .bicepparam files to simplify deployment scripts and improve maintainability. This eliminates manual parameter string construction and makes parameter management cleaner.
bicep build-paramscommandpxp-stagetopxp-stage-globalto clarify these are globally shared resourcesCLUSTER_RESOURCE_GROUP_NAMEvsGLOBAL_RESOURCE_GROUP_NAMEDownstream projects
Create new global resource group and move resources
The global resource group has been renamed from
{uniquePrefix}-{environment}to{uniquePrefix}-{environment}-global(e.g.,ppdemo-stage→ppdemo-stage-global) for better clarity. Resources keep their original names (e.g.,ppdemo-stage).{uniquePrefix}-{environment}-globalin Azure Portalppdemostage)ppdemo-stage)ppdemo-stage)ppdemo-stage-failure-anomalies)Update self-contained system version exports
In
cloud-infrastructure/cluster/deploy-cluster.sh, update the resource group variable name in your self-contained system version export (all downstream projects have at least one self-contained system beyond Account Management and Back Office):Add revisionSuffix parameter to container app modules
In
cloud-infrastructure/cluster/main-cluster.bicep, add therevisionSuffixparameter to your self-contained system container app modules:userAssignedIdentityName: yourSelfContainedSystemIdentityName ingress: true + revisionSuffix: revisionSuffix environmentVariables: yourSelfContainedSystemEnvironmentVariables(Optional) Migrate custom API keys and secrets to bicepparam
Only needed if you have custom API keys, client secrets, or service configurations beyond standard PlatformPlatform.
In
cloud-infrastructure/cluster/deploy-cluster.sh, ensure custom secrets are exported:In
cloud-infrastructure/cluster/main-cluster.bicepparam, add entries for custom parameters that were in the inline DEPLOYMENT_PARAMETERS string:Checklist