Skip to content

Remove use of storage account read tokens#32779

Merged
PureWeen merged 1 commit intodotnet:mainfrom
mmitche:remove-storage-account-read-tokens
Nov 20, 2025
Merged

Remove use of storage account read tokens#32779
PureWeen merged 1 commit intodotnet:mainfrom
mmitche:remove-storage-account-read-tokens

Conversation

@mmitche
Copy link
Member

@mmitche mmitche commented Nov 20, 2025

Variable group has nothing valid in it.

Copilot AI review requested due to automatic review settings November 20, 2025 15:20
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Nov 20, 2025
@dotnet-policy-service
Copy link
Contributor

Hey there @@mmitche! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

Copy link
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 removes references to an obsolete variable group "DotNetBuilds storage account read tokens" from Azure DevOps pipeline configuration files. According to the description, this variable group no longer contains valid credentials and should not be referenced.

Key Changes:

  • Removed the obsolete variable group reference from three pipeline YAML files

Reviewed Changes

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

File Description
eng/pipelines/maui-release-internal.yml Removed the DotNetBuilds storage account read tokens variable group reference from the variables section
eng/pipelines/common/variables.yml Removed the DotNetBuilds storage account read tokens variable group reference from the conditional variables block
eng/pipelines/ci-official.yml Removed the DotNetBuilds storage account read tokens variable group reference from the variables section

@PureWeen
Copy link
Member

/azp run MAUI-public

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@PureWeen
Copy link
Member

  • device test pipelines were able to run on essential pipelines

@PureWeen PureWeen merged commit 678e59c into dotnet:main Nov 20, 2025
7 of 11 checks passed
Copilot AI added a commit that referenced this pull request Nov 20, 2025
Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>
PureWeen added a commit that referenced this pull request Nov 20, 2025
…#32781)

* Initial plan

* Backport PRs #32610, #32694, #32779 - pipeline configuration fixes

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Jan 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants