Skip to content

fix(cd): require staging infra success before production deployment#377

Merged
rvanmaanen merged 3 commits intomainfrom
copilot/fix-bicep-production-deployment
May 1, 2026
Merged

fix(cd): require staging infra success before production deployment#377
rvanmaanen merged 3 commits intomainfrom
copilot/fix-bicep-production-deployment

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

  • Investigate actual CI failure - root cause is multi-line single-quoted string in openai.bicep line 25-26 (invalid Bicep syntax)
  • Fix: Make the @description a single-line string (single-quoted strings cannot span lines in Bicep)
  • Fix: Add deploy-staging-infra to production needs so production won't deploy if staging fails
  • Validate Bicep compiles successfully with az bicep build

Add deploy-staging-infra to the deploy-production job's needs array so
production deployment won't proceed if staging infrastructure fails.

Agent-Logs-Url: https://github.com/techhubms/techhub/sessions/946cdf93-0f88-4d45-b713-1c8edfa22724

Co-authored-by: rvanmaanen <4835258+rvanmaanen@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix staging failure causing production deployment issues fix(cd): require staging infra success before production deployment May 1, 2026
Copilot AI requested a review from rvanmaanen May 1, 2026 13:30
The @description decorator for adminIpAddresses used a single-quoted
string spanning two lines, which is invalid Bicep syntax. Single-quoted
strings cannot contain newlines. This caused the parser to fail on
every subsequent line, breaking both staging and production deployments.

Agent-Logs-Url: https://github.com/techhubms/techhub/sessions/0d549980-361d-4da0-b86b-84623d884815

Co-authored-by: rvanmaanen <4835258+rvanmaanen@users.noreply.github.com>
@rvanmaanen rvanmaanen marked this pull request as ready for review May 1, 2026 13:36
@rvanmaanen rvanmaanen requested a review from rajbos as a code owner May 1, 2026 13:36
@rvanmaanen rvanmaanen merged commit e47ec46 into main May 1, 2026
2 checks passed
@rvanmaanen rvanmaanen deleted the copilot/fix-bicep-production-deployment branch May 1, 2026 13:36
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

✅ PR Preview Environment Removed

The preview environment for this pull request has been removed.

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.

Bicep broken AND production deployment triggers when staging fails!

2 participants