Skip to content

[release/v7.4.20] Add the needed variable groups to the MSIX VPack pipeline to enforce using the production ADO feed - #27991

Merged
Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.4.20from
daxian-dbw:backport/release/v7.4.20/27896-ea61eef28
Sep 8, 2026
Merged

Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.4.20from
daxian-dbw:backport/release/v7.4.20/27896-ea61eef28

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

Backport of #27896 to release/v7.4.20

Triggered by Dongbo Wang (@daxian-dbw) on behalf of Dongbo Wang (@daxian-dbw)

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Adds the missing variable groups to the MSIX VPack pipeline so the insert-nuget-config-azfeed.yml template's "Switch to production Azure DevOps feed" step runs, granting permission to pull packages from upstream. Needed because the early-access .NET feed work changed the Coordinate Binary build to use a different internal feed, so downstream MSIX VPack builds now need authenticated access to pull packages.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

Introduced by the early-access .NET feed work (e.g., in v7.6.5), which changed the Coordinate Binary build to use a different internal feed, causing the MSIX VPack pipeline to fail without authenticated access to pull packages from upstream.

Testing

Pipeline configuration change; verified original PR fixed the MSIX VPack pipeline failure on master/downstream builds.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Modifies release build pipeline configuration (variable groups/feed authentication) — build infra risk — but the change is narrowly scoped to restoring a working feed-authentication step and is necessary to unblock MSIX VPack builds on this release line.

Copilot AI lite review requested due to automatic review settings September 8, 2026 21:07
@daxian-dbw
Dongbo Wang (daxian-dbw) requested a review from a team as a code owner September 8, 2026 21:07
@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 Sep 8, 2026
@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.

🟢 Approval recommended

The change is narrowly scoped to adding the required variable groups and aligns with the existing pipeline template’s conditional feed-switching behavior.

Pull request overview

This PR backports the master fix to the release/v7.4.20 branch by updating the MSIX VPack official pipeline to include the Azure DevOps feed variable groups required for authenticated upstream package pulls.

Changes:

  • Adds the mscodehub-feed-read-general and mscodehub-feed-read-akv variable groups to .pipelines/MSIXBundle-vPack-Official.yml so the insert-nuget-config-azfeed.yml template can run its “Switch to production Azure DevOps feed…” step when UseAzDevOpsFeed is set.
File summaries
File Description
.pipelines/MSIXBundle-vPack-Official.yml Adds missing variable groups needed for ADO feed authentication in the MSIX VPack pipeline.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 917d350 into PowerShell:release/v7.4.20 Sep 8, 2026
36 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the backport/release/v7.4.20/27896-ea61eef28 branch September 8, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants