Repository navigation
Use a deployment job for PCS releases - #6799
Merged
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
🔵 Needs a closer look
Azure DevOps/1ES server-side validation and a staging deployment remain pending for this production release pipeline change.
No open issues 🎉
What changed in this PR
Consolidates PCS releases into one environment-gated stage while preserving the existing deployment workflow.
Changes:
- Runs database migrations before the
DeployPCSdeployment job in a singleReleasestage. - Adds explicit checkout and selective artifact downloads; updates the E2E dependency.
- Documents environment approvals and the existing CLI-based rollout.
| File | Description |
|---|---|
| eng/pipelines/templates/stages/deploy.yaml | Consolidates release jobs and updates E2E sequencing. |
| docs/DevGuide.md | Explains release gating and corrects the deployment CLI link. |
Review effort: Balanced
Declare same-run pipeline artifacts through templateContext inputs and document the resource-less environment requirement. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove checkout from the release job and include SDK configuration and the GitHub App token helper in the CLI artifact. Keep source-based helper defaults for existing non-release callers. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Member
Author
Member
|
We intentionally separated DB migration and Deployment into different sequential stages so we can just run deployment in case of a Maestro outage where we need to redeploy some version. Is there a way to separate them while keeping the deployment job? |
premun
enabled auto-merge (squash)
October 21, 2026 13:56
dkurepa
approved these changes
Oct 21, 2026
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
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
Releasestage.DeployPCSrunOncedeployment job, retaining the existingProduction/Stagingenvironment selection and approval protection for both jobs.DeployPCSas a 1ESreleaseJob, withisProductiontrue only onrefs/heads/production.global.jsonand the GitHub App token helper in the CLI artifact, and declare same-run PCS CLI and production-onlyAssetManifestsartifacts throughtemplateContext.inputs.None(no attached resources).No changes to Azure DevOps environment approval settings or exclusive locking are included.
Azure DevOps/1ES server-side template validation and a staging deployment remain to be performed; authenticated Azure DevOps access was unavailable locally. Confirm that both the
ProductionandStagingenvironments use resource typeNonebefore running the deployment.