Skip to content

[release/v7.4.20] Fix install dotnet for non early access release - #27926

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

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

Conversation

@adityapatwardhan

Copy link
Copy Markdown
Member

Backport of #27892 to release/v7.4.20

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.4.20. 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:34
@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

Backports pipeline template updates to release/v7.4.20 to ensure release validation jobs initialize version variables early and avoid forcing non-early-access .NET installation into a non-default OneBranch restore phase configuration.

Changes:

  • Insert SetVersionVariables.yml immediately after checkout so RepoRoot/version variables are initialized before subsequent release-tag/container setup.
  • Update install-dotnet.yml invocation to set ob_restore_phase: false only for early-access builds (non-early-access uses 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; gates ob_restore_phase: false to early-access .NET install path.
.pipelines/templates/release-validate-fxdpackages.yml Same adjustments as globaltools for FXD package validation jobs.

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

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit a7fec0a into PowerShell:release/v7.4.20 Sep 1, 2026
36 checks passed
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) deleted the backport/release/v7.4.20/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