Skip to content

Add the needed variable groups to the MSIX VPack pipeline to enforce using the production ADO feed - #27896

Merged
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
daxian-dbw:msix-vpack-fix
Aug 27, 2026
Merged

Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
daxian-dbw:msix-vpack-fix

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

PR Summary

Add the needed variable groups to the MSIX VPack pipeline to enforce using the production ADO feed.

The MSIX VPack pipeline is using the public ADO feed without PAT for building PowerShell. This works fine before because all needed packages are already pulled to the ADO feed from the upstream by the Coordinate Binary build, so later when running MSIX VPack pipeline, there is no need to pull a NuGet package from upstream, and accessing available NuGet packages in that feed doesn't require authentication.

But with the early-access .NET feed work, e.g. in PowerShell v7.6.5, the Coordinate Binary build uses a different internal feed for build and thus when it gets to the MSIX VPack pipeline, it requires the needed packages to be pulled from upstream to the ADO feed, which requires authentication, and thus the build failed.

This change will make sure the 'Switch to production Azure DevOps feed for all nuget.configs' step gets run from the insert-nuget-config-azfeed.yml template, to give the pipeline the permission to pull packages from upstream.

@daxian-dbw
Dongbo Wang (daxian-dbw) marked this pull request as ready for review August 27, 2026 22:01
Copilot AI lite review requested due to automatic review settings August 27, 2026 22:01
@daxian-dbw
Dongbo Wang (daxian-dbw) requested a review from a team as a code owner August 27, 2026 22:01
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

1 similar comment
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
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

Updates the MSIX VPack official pipeline to include the Azure DevOps feed-read variable groups required for authenticated access to the production ADO feed, ensuring the insert-nuget-config-azfeed.yml template can switch all nuget.config files to the private/production feed when UseAzDevOpsFeed is enabled.

Changes:

  • Add mscodehub-feed-read-general variable group to the pipeline.
  • Add mscodehub-feed-read-akv variable group to the pipeline.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Aug 27, 2026
@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit ea61eef into PowerShell:master Aug 27, 2026
38 of 39 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the msix-vpack-fix branch August 27, 2026 22:06
@daxian-dbw

Copy link
Copy Markdown
Member Author

7.6.5 release branch in ADO already has this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.4.x-Done Backport-7.5.x-Consider CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants