Skip to content

fix: key vault#30

Merged
dewitt4 merged 1 commit into
mainfrom
fix/pipeline
Jun 23, 2026
Merged

fix: key vault#30
dewitt4 merged 1 commit into
mainfrom
fix/pipeline

Conversation

@dewitt4

@dewitt4 dewitt4 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

infra/main.bicep — new pipelineSpOid param (defaults to ''); new keyVaultRes existing reference using the pre-computed keyVaultName variable; conditional Key Vault Secrets Officer role assignment scoped to it infra/main.parameters.staging.json — pipelineSpOid set to the SP OID from the error log infra/main.parameters.prod.json — same SP OID (assuming the same service connection is used for prod; change if prod uses a different SP)

@dewitt4 dewitt4 self-assigned this Jun 23, 2026
@dewitt4 dewitt4 added the bug Something isn't working label Jun 23, 2026
infra/main.bicep — new pipelineSpOid param (defaults to ''); new keyVaultRes existing reference using the pre-computed keyVaultName variable; conditional Key Vault Secrets Officer role assignment scoped to it
infra/main.parameters.staging.json — pipelineSpOid set to the SP OID from the error log
infra/main.parameters.prod.json — same SP OID (assuming the same service connection is used for prod; change if prod uses a different SP)
@dewitt4
dewitt4 merged commit b240d7c into main Jun 23, 2026
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant