Skip to content

Pin Linux image version to Ubuntu 24.04 for CI tests - #28307

Merged
Aditya Patwardhan (adityapatwardhan) merged 5 commits into
masterfrom
FixLinuxCI
Nov 3, 2026
Merged

Aditya Patwardhan (adityapatwardhan) merged 5 commits into
masterfrom
FixLinuxCI

Conversation

@adityapatwardhan

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

Copilot AI balanced review requested due to automatic review settings November 3, 2026 18:36
@azure-pipelines

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

@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 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.

🟡 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
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

Comment thread .github/workflows/linux-ci.yml Resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants