- Notifications
You must be signed in to change notification settings - Fork8.1k
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
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.
Uh oh!
There was an error while loading.Please reload this page.
/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI |
| Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
f44c11c intoPowerShell:masterUh oh!
There was an error while loading.Please reload this page.
microsoft-github-policy-servicebot commentedMar 27, 2025 • edited by unfurl-linksbot
Loading Uh oh!
There was an error while loading.Please reload this page.
edited by unfurl-linksbot
Uh oh!
There was an error while loading.Please reload this page.
📣 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. |
JustinGrote commentedApr 2, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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. |
JustinGrote commentedApr 3, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@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. |
Uh oh!
There was an error while loading.Please reload this page.
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.