Skip to content

Remove dead DotNet-VSTS-Infra-Access variable group reference#16203

Merged
nohwnd merged 1 commit into
microsoft:mainfrom
missymessa:cleanup/remove-dead-vg-reference
Jul 3, 2026
Merged

Remove dead DotNet-VSTS-Infra-Access variable group reference#16203
nohwnd merged 1 commit into
microsoft:mainfrom
missymessa:cleanup/remove-dead-vg-reference

Conversation

@missymessa

Copy link
Copy Markdown
Contributor

The \DotNet-VSTS-Infra-Access\ variable group (VG 4 in dnceng/internal) only contained the \dn-bot-devdiv-drop-r-code-r\ PAT which has been fully migrated to WIF:

  • WIF service connection: \dnceng-devdiv-drop-rw-code-rw-wif\ (already in use by this pipeline, as noted in the remaining comment)
  • PAT status: Expired (May 31, 2026) and disabled in Key Vault
  • Secret manifest: Removed from .vault-config/product-builds-engkeyvault.yaml\

The variable group is empty/stale and its reference is dead code. The _DevDivDropAccessToken\ is already set to ''\ with WIF handling the token acquisition.

Related: https://dev.azure.com/dnceng/internal/_workitems/edit/10147

The DotNet-VSTS-Infra-Access variable group (VG 4 in dnceng/internal)
only contained the dn-bot-devdiv-drop-r-code-r PAT which has been
migrated to WIF. The PAT is expired and disabled. The variable group
reference is no longer needed since DevDiv drop access token is now
acquired via WIF service connection.
Copilot AI review requested due to automatic review settings July 2, 2026 19:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes a stale Azure DevOps variable group reference from the official pipeline now that DevDiv drop authentication is handled via a WIF service connection rather than a PAT-backed variable group.

Changes:

  • Removed the DotNet-VSTS-Infra-Access variable group reference from azure-pipelines-official.yml.

@nohwnd nohwnd enabled auto-merge (squash) July 3, 2026 08:02
@nohwnd nohwnd merged commit fdd72c5 into microsoft:main Jul 3, 2026
17 checks passed
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