Skip to content

[release/v7.5.11] Fix install dotnet for non early access release - #27925

Merged
Aditya Patwardhan (adityapatwardhan) merged 1 commit into
PowerShell:release/v7.5.11from
adityapatwardhan:backport/release/v7.5.11/27892-9fa54b610
Sep 1, 2026
Merged

Aditya Patwardhan (adityapatwardhan) merged 1 commit into
PowerShell:release/v7.5.11from
adityapatwardhan:backport/release/v7.5.11/27892-9fa54b610

Conversation

@adityapatwardhan

Copy link
Copy Markdown
Member

Backport of #27892 to release/v7.5.11

Triggered by Aditya Patwardhan (@adityapatwardhan) on behalf of Aditya Patwardhan (@adityapatwardhan)

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)

Corrects release validation pipeline setup by initializing version variables before release tag/container setup and limiting ob_restore_phase: false to early-access builds, so non-early-access release validation installs .NET with the intended defaults.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Cherry-pick completed cleanly against the latest fetched upstream/release/v7.5.11. Verified the delta contains only the two intended release validation templates, git diff --check passes, tracked files contain no conflict markers, Prettier 3.6.2 parses both changed YAML files successfully, and the worktree is clean.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

This changes release validation pipelines for framework-dependent packages and global tools across release builds. The change is narrowly scoped to two template invocations and was validated for YAML syntax and exact diff content, but pipeline changes can affect release validation on multiple platforms.

Copilot AI lite review requested due to automatic review settings September 1, 2026 01:33
@adityapatwardhan Aditya Patwardhan (adityapatwardhan) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Sep 1, 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.

Pull request overview

This PR backports #27892 to release/v7.5.11 to fix release validation pipeline behavior for non-early-access releases, specifically around when version variables are initialized and when install-dotnet.yml is forced into ob_restore_phase: false.

Changes:

  • Add SetVersionVariables.yml early in both release-validate-fxdpackages and release-validate-globaltools jobs to initialize release/version variables before subsequent release-tag/container setup.
  • Update install-dotnet.yml invocations to only pass ob_restore_phase: false when IsEarlyAccess is true (so non-early-access runs use the template’s default behavior).

Reviewed changes

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

File Description
.pipelines/templates/release-validate-globaltools.yml Initializes version variables earlier and gates ob_restore_phase: false behind IsEarlyAccess for dotnet install.
.pipelines/templates/release-validate-fxdpackages.yml Same adjustments as globaltools validation: early version initialization and early-access-only ob_restore_phase: false.

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

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit 8e9bfe8 into PowerShell:release/v7.5.11 Sep 1, 2026
35 checks passed
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) deleted the backport/release/v7.5.11/27892-9fa54b610 branch September 1, 2026 01:40
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.

3 participants