Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Merged
TravisEz13 merged 2 commits intoPowerShell:masterfromTravisEz13:ubuntu
Mar 27, 2025
Merged

Conversation

@TravisEz13
Copy link
Member

@TravisEz13TravisEz13 commentedMar 27, 2025
edited
Loading

This pull request includes updates to the.github/workflows/linux-ci.yml file to ensure the CI builds use the latest Ubuntu version. The most important changes involve modifying theruns-on parameter for several jobs.

Changes to CI build configuration:

  • Updatedruns-on parameter fromubuntu-20.04 toubuntu-latest for theci_build job. (.github/workflows/linux-ci.yml)
  • Updatedruns-on parameter fromubuntu-20.04 toubuntu-latest for the job that depends onci_build andchanges. (.github/workflows/linux-ci.yml)[1][2][3][4]

Updates 2025-04-02

Context

GitHub is removing support for20.04 and was having brownouts at the time of this PR to force migration.

@TravisEz13TravisEz13 requested a review froma team as acode ownerMarch 27, 2025 19:14
@TravisEz13TravisEz13 added the CL-BuildPackagingIndicates that a PR should be marked as a build or packaging change in the Change Log labelMar 27, 2025
@TravisEz13TravisEz13 requested review fromCopilot and removed request fora teamMarch 27, 2025 19:14
Copy link
Contributor

CopilotAI left a 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.

@TravisEz13
Copy link
MemberAuthor

/azp run PowerShell-CI-linux-packaging, PowerShell-Windows-Packaging-CI

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@TravisEz13TravisEz13enabled auto-merge (squash)March 27, 2025 20:32
@TravisEz13TravisEz13 merged commitf44c11c intoPowerShell:masterMar 27, 2025
36 of 37 checks passed
@TravisEz13TravisEz13 deleted the ubuntu branchMarch 27, 2025 20:33
@microsoft-github-policy-service
Copy link
Contributor

microsoft-github-policy-servicebot commentedMar 27, 2025
edited by unfurl-linksbot
Loading

📣 Hey@TravisEz13, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗https://aka.ms/PSRepoFeedback

@JustinGrote
Copy link
Contributor

@TravisEz13 I missed this one, aren't you concerned about CI suddenly breaking if latest changes? Shouldn't this stay as a pinned version?

@kilasuit
Copy link
Collaborator

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

@TravisEz13

This comment was marked as outdated.

@TravisEz13
Copy link
MemberAuthor

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.

@TravisEz13
Copy link
MemberAuthor

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 reacted with thumbs up emoji

@JustinGrote
Copy link
Contributor

JustinGrote commentedApr 2, 2025
edited
Loading

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
https://github.blog/changelog/2025-02-12-notice-of-upcoming-deprecations-and-breaking-changes-for-github-actions/
actions/runner-images#11101

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 to24.04 if possible,22.04 if that is all that is supported for now.

@iSazonov
Copy link
Collaborator

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
Copy link
Contributor

JustinGrote commentedApr 3, 2025
edited
Loading

@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 sayslibso4.so.dll not found) that might be way hard to figure out until you realize the distro bumped. Rollouts like these come with no specific notice either as they come in rings, whereas the team can control for issues by simply PRing a version bump to the github action and seeing how it does on that branch before merging. You know, like every other change made to the repo.

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.

iSazonov reacted with thumbs up emoji

Sysoiev-Yurii pushed a commit to Sysoiev-Yurii/PowerShell that referenced this pull requestMay 12, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

Copilot code reviewCopilotCopilot left review comments

@jshigetomijshigetomijshigetomi approved these changes

Assignees

No one assigned

Labels

BackPort-7.4.x-DoneBackport-7.5.x-MigratedCL-BuildPackagingIndicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@TravisEz13@JustinGrote@kilasuit@iSazonov@jshigetomi

[8]ページ先頭

©2009-2025 Movatter.jp