Skip to content

[release/8.0.1xx] Use MI in VMR sync pipeline instead of an AzDO PAT#20869

Merged
premun merged 2 commits into
dotnet:release/8.0.1xxfrom
premun:prvysoky/mi
May 18, 2026
Merged

[release/8.0.1xx] Use MI in VMR sync pipeline instead of an AzDO PAT#20869
premun merged 2 commits into
dotnet:release/8.0.1xxfrom
premun:prvysoky/mi

Conversation

@premun
Copy link
Copy Markdown
Member

@premun premun commented May 13, 2026

Copilot AI review requested due to automatic review settings May 13, 2026 11:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the VMR synchronization pipeline to stop using an Azure DevOps PAT variable group and instead use a federated (WIF) token when syncing internal VMR branches, aligning with the move toward managed identity / federated auth.

Changes:

  • Switch vmr-sync.sh invocation to use $(AzdoToken) instead of the dn-bot-all-orgs-code-r PAT variable.
  • Remove the DotNetBot-AzDO-PAT variable group reference from the internal/release job path.
  • Add a step to mint an AzDO access token via /eng/common/templates-official/steps/get-federated-access-token.yml (outputting AzdoToken).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
eng/pipelines/templates/steps/vmr-pull-updates.yml Updates the vmr-sync.sh call to use $(AzdoToken) for --azdev-pat.
eng/pipelines/templates/jobs/vmr-synchronization.yml Removes PAT variable group usage and adds a federated-token minting step for internal/release syncs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread eng/pipelines/templates/jobs/vmr-synchronization.yml Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread eng/pipelines/templates/steps/vmr-pull-updates.yml
Comment thread eng/pipelines/templates/jobs/vmr-synchronization.yml
@premun premun merged commit 31d3021 into dotnet:release/8.0.1xx May 18, 2026
22 checks passed
@premun premun deleted the prvysoky/mi branch May 18, 2026 13:40
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.

3 participants