Skip to content

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

Merged
wtgodbe merged 2 commits into
release/8.0from
pat-migration/wif-mirror-within-azdo-release8
Apr 8, 2026
Merged

[release/8.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo#66115
wtgodbe merged 2 commits into
release/8.0from
pat-migration/wif-mirror-within-azdo-release8

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/8.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 review from a team and wtgodbe as code owners April 2, 2026 17:56
@missymessa missymessa added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 2, 2026
Copilot AI review requested due to automatic review settings April 2, 2026 17:56
@dotnet-policy-service dotnet-policy-service Bot added this to the 8.0.x milestone Apr 2, 2026
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

This PR migrates the internal AzDO “mirror-within-azdo” pipeline from using a stored PAT (via the Mirror-Credentials variable group) to using a Workload Identity Federation (WIF) service connection, aligning the release/8.0 branch with the already-merged change on main.

Changes:

  • Removed the Mirror-Credentials variable group dependency.
  • Added an AzureCLI@2 step to mint an AzDO bearer token via az account get-access-token using the dnceng-build-rw-code-rw-wif service connection and store it as a secret variable.
  • Updated git clone and git push to use header-based auth (http.https://dev.azure.com/.extraheader) instead of embedding credentials in the URL.

@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/8.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo Apr 8, 2026
@wtgodbe wtgodbe merged commit 3b1b00d into release/8.0 Apr 8, 2026
25 checks passed
@wtgodbe wtgodbe deleted the pat-migration/wif-mirror-within-azdo-release8 branch April 8, 2026 21:07
@dotnet-policy-service dotnet-policy-service Bot modified the milestones: 8.0.x, 8.0.27 Apr 8, 2026
This was referenced May 16, 2026
HriplyiPlintus pushed a commit to HriplyiPlintus/booking-app that referenced this pull request May 16, 2026
Updated
[Microsoft.AspNetCore.Mvc.Testing](https://github.com/dotnet/aspnetcore)
from 8.0.26 to 8.0.27.

<details>
<summary>Release notes</summary>

_Sourced from [Microsoft.AspNetCore.Mvc.Testing's
releases](https://github.com/dotnet/aspnetcore/releases)._

## 8.0.27

[Release](https://github.com/dotnet/core/releases/tag/v8.0.27)

## What's Changed
* [release/8.0] Update branding to 8.0.27 by @​vseanreesermsft in
dotnet/aspnetcore#66205
* [release/8.0] Update NPM dependencies by @​wtgodbe in
dotnet/aspnetcore#66052
* [release/8.0] Move off of dead-lettered Windows preview helix queue by
@​wtgodbe in dotnet/aspnetcore#66220
* [release/8.0] (deps): Bump src/submodules/googletest from `73a63ea` to
`d72f9c8` by @​dependabot[bot] in
dotnet/aspnetcore#66087
* [release/8.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF
service connection in mirror-within-azdo by @​missymessa in
dotnet/aspnetcore#66115
* [release/8.0] Update dependencies from dotnet/source-build-externals
by @​dotnet-maestro[bot] in
dotnet/aspnetcore#66216
* [release/8.0] Update dependencies from dotnet/arcade by
@​dotnet-maestro[bot] in dotnet/aspnetcore#66081
* [release/8.0] Update dependencies from
dotnet/source-build-reference-packages by @​dotnet-maestro[bot] in
dotnet/aspnetcore#66131
* [release/8.0] Update @​azure/msal-browser from 2.x to 4.x by @​wtgodbe
in dotnet/aspnetcore#66236
* [release/8.0] Use source-build-assets repo by @​NikolaMilosavljevic in
dotnet/aspnetcore#66276
* Merging internal commits for release/8.0 by @​vseanreesermsft in
dotnet/aspnetcore#66317


**Full Changelog**:
dotnet/aspnetcore@v8.0.26...v8.0.27

Commits viewable in [compare
view](dotnet/aspnetcore@v8.0.26...v8.0.27).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Microsoft.AspNetCore.Mvc.Testing&package-manager=nuget&previous-version=8.0.26&new-version=8.0.27)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants