-
Notifications
You must be signed in to change notification settings - Fork 8k
Switch to ubuntu-lastest for CI #25247
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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.
Pull Request Overview
This PR updates the CI build configuration to use the latest Ubuntu image for all relevant jobs.
- Changed the "runs-on" parameter from "ubuntu-20.04" to "ubuntu-latest" for the ci_build job and for dependent jobs.
|
/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI |
|
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
|
📣 Hey @TravisEz13, how did we do? We would love to hear your feedback with the link below! 🗣️ 🔗 https://aka.ms/PSRepoFeedback |
|
@TravisEz13 I missed this one, aren't you concerned about CI suddenly breaking if latest changes? Shouldn't this stay as a pinned version? |
|
I agree with @JustinGrote here. We need to understand the reasoning for this chnage, which should have been added as Context to this pr, which is missing |
This comment was marked as outdated.
This comment was marked as outdated.
|
The environment is continuously changing. We can pin again if we are broken. 20.04 is currently in brown out. This was a required change to keep the CI functioning. We can pin to 22.04 if needed. I would prefer to see the changes to the environment and be aware than discover it when we are forced to move again. |
|
I'll try to include the context, but I was in a rush to get the system working again during the 20.04 brown out. |
|
Reasonable, it's not like the 20.04 brown out was a secret though. https://github.blog/changelog/2025-01-15-github-actions-ubuntu-20-runner-image-brownout-dates-and-other-breaking-changes/#ubuntu-20-image-is-closing-down It's my opinion that it is better to be potentially broken (assuming the notice isn't observed in the future) by a well known deprecation that provides a well known error message when it happens, than to be broken by some unknown change in a "latest" update that may be difficult to troubleshoot, or even know that a rev happened, so I vote for the pin to |
|
What is the problem? That there may be versions of Ubuntu that we will never support? Or is it that something might break at the most inopportune moment? Wouldn't it be easiest to just create optional CI with the ubuntu-latest version? This will immediately flag the change. |
|
@iSazonov I'm concerned about the latter, that suddenly ubuntu-latest becomes 26.04 and it breaks the build in a non-specific way (e.g. error that says I agree that doing a matrix with both ubuntu-latest and a pinned version would also likely (somewhat) clearly indicate the issue, so that is also acceptable. |
This pull request includes updates to the
.github/workflows/linux-ci.ymlfile to ensure the CI builds use the latest Ubuntu version. The most important changes involve modifying theruns-onparameter for several jobs.Changes to CI build configuration:
runs-onparameter fromubuntu-20.04toubuntu-latestfor theci_buildjob. (.github/workflows/linux-ci.yml)runs-onparameter fromubuntu-20.04toubuntu-latestfor the job that depends onci_buildandchanges. (.github/workflows/linux-ci.yml) [1] [2] [3] [4]Updates 2025-04-02
Context
GitHub is removing support for
20.04and was having brownouts at the time of this PR to force migration.