Skip to content

Fix internal auth#55305

Merged
ellahathaway merged 2 commits into
dotnet:release/9.0.1xxfrom
ellahathaway:fix-internal-auth
Jul 17, 2026
Merged

Fix internal auth#55305
ellahathaway merged 2 commits into
dotnet:release/9.0.1xxfrom
ellahathaway:fix-internal-auth

Conversation

@ellahathaway

@ellahathaway ellahathaway commented Jul 15, 2026

Copy link
Copy Markdown
Member

Backport dotnet/dotnet#7697 to release/9.0.1xx

Copilot AI review requested due to automatic review settings July 15, 2026 21:11
@ellahathaway
ellahathaway requested review from a team as code owners July 15, 2026 21:11
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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

Backports dotnet/dotnet#7697 to the release/8.0.1xx branch by updating the VMR build pipeline template to use OAuth-based authentication (via the pipeline access token) instead of a retired PAT/variable group when preparing internal NuGet feed credentials for smoke tests.

Changes:

  • Remove use of the AzureDevOps-Artifact-Feeds-Pats variable group for internal runs.
  • Switch the smoke-test NuGet credential injection from $(dn-bot-dnceng-artifact-feeds-rw) to $(System.AccessToken) with updated inline rationale comments.

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

@ellahathaway
ellahathaway enabled auto-merge (squash) July 15, 2026 21:31
@ellahathaway

Copy link
Copy Markdown
Member Author

/azp run sdk-source-build

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@ellahathaway

Copy link
Copy Markdown
Member Author

@nagilson - the PR label checks are failing. This PR is required for servicing. Can you help with getting the correct labels on this PR so that those checks pass? Thanks

@ellahathaway
ellahathaway merged commit 18761f0 into dotnet:release/9.0.1xx Jul 17, 2026
36 of 38 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.

4 participants