Skip to content

[release/10.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo#66113

Merged
wtgodbe merged 3 commits into
release/10.0from
pat-migration/wif-mirror-within-azdo-release10
Apr 8, 2026
Merged

[release/10.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo#66113
wtgodbe merged 3 commits into
release/10.0from
pat-migration/wif-mirror-within-azdo-release10

Conversation

@missymessa
Copy link
Copy Markdown
Member

Summary

Migrate the azure-pipelines-mirror-within-azdo.yml pipeline from using the dn-bot-dnceng-build-rw-code-rw PAT (from the Mirror-Credentials variable group) to the dnceng-build-rw-code-rw-wif Workload Identity Federation service connection.

This is the same change as #66074 (merged to main), ported to the release/10.0 branch.

Changes

  • Remove Mirror-Credentials variable group reference
  • New AzureCLI@2 step – mints an AzDO bearer token via az account get-access-token using the dnceng-build-rw-code-rw-wif WIF service connection and stores it as the secret pipeline variable WifAzdoToken
  • Clone step now uses header-based auth (http.extraheader) instead of PAT embedded in the URL
  • Push step now uses header-based auth as well

Related

  • Part of PAT migration work item WI 10139
  • Service connection: dnceng-build-rw-code-rw-wif (Entra app 21f66e0-bb35-4fd3-bc70-ba084d1e7a52)

@missymessa missymessa requested a review from wtgodbe as a code owner April 2, 2026 17:23
@missymessa missymessa added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 2, 2026
@missymessa missymessa requested a review from a team as a code owner April 2, 2026 17:23
Copilot AI review requested due to automatic review settings April 2, 2026 17:23
Copy link
Copy Markdown
Contributor

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

Migrates the azure-pipelines-mirror-within-azdo.yml pipeline off the dn-bot-dnceng-build-rw-code-rw PAT and onto the dnceng-build-rw-code-rw-wif Workload Identity Federation (WIF) service connection for authenticating Git operations against Azure DevOps.

Changes:

  • Removed the Mirror-Credentials variable group reference (PAT no longer required).
  • Added an AzureCLI@2 step to mint an Azure DevOps AAD access token and store it as a secret pipeline variable (WifAzdoToken).
  • Updated git clone and git push to use http.extraheader with the minted bearer token.

@missymessa
Copy link
Copy Markdown
Member Author

/azp run aspnetcore-ci

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wtgodbe wtgodbe changed the title Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo [release/9.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo Apr 8, 2026
@wtgodbe wtgodbe changed the title [release/9.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo [release/10.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo Apr 8, 2026
@wtgodbe
Copy link
Copy Markdown
Member

wtgodbe commented Apr 8, 2026

Failure unrelated

@wtgodbe wtgodbe merged commit a3e9e1b into release/10.0 Apr 8, 2026
25 of 28 checks passed
@wtgodbe wtgodbe deleted the pat-migration/wif-mirror-within-azdo-release10 branch April 8, 2026 23:36
@wtgodbe wtgodbe added the tell-mode Indicates a PR which is being merged during tell-mode label Apr 8, 2026
@dotnet-policy-service dotnet-policy-service Bot added this to the 10.0.7 milestone Apr 8, 2026
@wtgodbe wtgodbe modified the milestones: 10.0.7, 10.0.8 Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants