GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Pin Linux image version to Ubuntu 24.04 for CI tests by adityapatwardhan · Pull Request #28307 · PowerShell/PowerShell

Conversation

@adityapatwardhan

@adityapatwardhan Aditya Patwardhan (adityapatwardhan) commented Nov 3, 2026

Copy link
Copy Markdown
Member

PR Summary

Changing the version from Ubuntu 24 -> Ubuntu 26 causes error in CI.

  sudo: preserving the entire environment is not supported, '-E' is ignored

PR Context

PR Checklist

  • PR has a meaningful title
    • Use the present tense and imperative mood when describing your changes
  • Summarized changes
  • Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
  • This PR is ready to merge. If this PR is a work in progress, please open this as a Draft Pull Request and mark it as Ready to Review when it is ready to merge.
  • Breaking changes
    • None
    • OR
    • Experimental feature(s) needed
      • Experimental feature name(s):
  • User-facing changes
    • Not Applicable
    • OR
    • Documentation needed
      • Issue filed:
  • Testing - New and feature
    • N/A or can only be tested interactively
    • OR
    • Make sure you've added a new test if existing tests do not effectively test the code changed

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@adityapatwardhan
Pin Linux image version to Ubuntu 24.04 for CI tests
237a9f8
Copilot AI balanced review requested due to automatic review settings November 3, 2026 18:36
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) requested review from a team and Justin Chung (jshigetomi) as code owners November 3, 2026 18:36
@azure-pipelines

azure-pipelines Bot commented Nov 3, 2026

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@adityapatwardhan
Fix bad merge
78c5384
anamnavi
Anam Navied (anamnavi) approved these changes Nov 3, 2026
View reviewed changes
Copilot started reviewing on behalf of Aditya Patwardhan (adityapatwardhan) November 3, 2026 18:42
View session
@adityapatwardhan Aditya Patwardhan (adityapatwardhan) added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Nov 3, 2026
Copilot AI reviewed Nov 3, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The added ci_build dependency causes the Linux packaging job to be skipped for packaging-only PRs, even though its action performs its own build.

1 open issue
  • Medium severity Remove ci_build dependency from packaging-only PRs · New
What changed in this PR

Pins Linux CI to Ubuntu 24.04 to avoid the reported sudo -E incompatibility when ubuntu-latest moves to Ubuntu 26.

Changes:

  • Replaces ubuntu-latest with ubuntu-24.04 throughout the Linux workflow, including the reusable xUnit workflow input.
  • Moves the Linux packaging job before the merge-readiness job and adds a ci_build dependency.
File Description
.github/​workflows/​linux-ci.yml Pins Ubuntu runners and updates the packaging job’s placement and dependencies.

Review effort: Balanced

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread .github/workflows/linux-ci.yml Resolved

Uh oh!

There was an error while loading. Please reload this page.
Aditya Patwardhan (adityapatwardhan) added 3 commits November 3, 2026 11:45
@adityapatwardhan
Fix bad merge 2
7c3a795
@adityapatwardhan
Remove extra line
bc8f8a1
@adityapatwardhan
Re-add removed line
08cc9da
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) enabled auto-merge (squash) November 3, 2026 19:46
TravisEz13
Travis Plunk (TravisEz13) approved these changes Nov 3, 2026
View reviewed changes
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit ffa8402 into master Nov 3, 2026
38 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@adityapatwardhan Aditya Patwardhan (adityapatwardhan) added Backport-7.6.x-Approved Backport-7.7.x-Approved labels Nov 3, 2026
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) deleted the FixLinuxCI branch November 3, 2026 20:51
@daxian-dbw Dongbo Wang (daxian-dbw) mentioned this pull request Nov 3, 2026
[release/v7.6.10] Pin Linux image version to Ubuntu 24.04 for CI tests #28312
Merged
3 of 9 tasks
@daxian-dbw Dongbo Wang (daxian-dbw) added Backport-7.6.x-Migrated Backport-7.6.x-Done and removed Backport-7.6.x-Approved Backport-7.6.x-Migrated labels Nov 3, 2026
This was referenced Nov 5, 2026
[release/v7.7.0] Pin Linux image version to Ubuntu 24.04 for CI tests #28351
Closed
[release/v7.7.0] Pin Linux image version to Ubuntu 24.04 for CI tests #28357
Merged
@jshigetomi Justin Chung (jshigetomi) added Backport-7.7.x-Migrated and removed Backport-7.7.x-Approved labels Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@TravisEz13 Travis Plunk (TravisEz13) TravisEz13 approved these changes
@jshigetomi Justin Chung (jshigetomi) Awaiting requested review from jshigetomi jshigetomi is a code owner
+1 more reviewer
@anamnavi Anam Navied (anamnavi) anamnavi approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

Backport-7.6.x-Done Backport-7.7.x-Migrated CL-Test Indicates that a PR should be marked as a test change in the Change Log

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

6 participants

@adityapatwardhan @TravisEz13 @anamnavi @daxian-dbw @jshigetomi
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.