-
-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Change base OS image to Ubuntu-24.04 in CI testing #122544
Comments
Thanks for the report, we're using ❯ rg ubuntu-22.04
.readthedocs.yml
11: os: ubuntu-22.04
.azure-pipelines/ci.yml
8: vmImage: ubuntu-22.04
.azure-pipelines/pr.yml
8: vmImage: ubuntu-22.04
20: vmImage: ubuntu-22.04
.github/workflows/build.yml
47: runs-on: ubuntu-22.04
198: runs-on: ubuntu-22.04
264: runs-on: ubuntu-22.04
378: runs-on: ubuntu-22.04
.github/workflows/reusable-wasi.yml
12: runs-on: ubuntu-22.04
.github/workflows/reusable-tsan.yml
22: runs-on: ubuntu-22.04
.github/workflows/reusable-ubuntu.yml
17: runs-on: ubuntu-22.04 Yes, let's upgrade, assuming all the dependencies and everything installs fine. Would you like to open some PRs? I suggest a separate PR for each of Read the Docs, Azure Pipelines and GitHub Actions. |
Sure!, Of course, I'd love to contribute to this, let me first investigate if it would affect the result of CI testing if just simply change |
We are also using Ubuntu 22.04 for generating the configure script in the Docker image but I don't think we should change this one. This is in |
The comment at the top says they should match: cpython/Tools/build/regen-configure.sh Lines 5 to 8 in fda6bd8
Which should be the same as this one: cpython/.github/workflows/build.yml Lines 43 to 47 in fda6bd8
|
Oh then we should update both of them then! (and I'll need to update my PR in the devguide as well) |
Some other
|
No, let's not change any In fact, https://github.com/actions/runner-images?tab=readme-ov-file#available-images says Do you know when GA is planned? |
According to there discussion in I think we can move all CI-testing step-by-step. |
…ythonGH-122568) (cherry picked from commit d57f8a9) Co-authored-by: Damien <[email protected]>
…ythonGH-122568) (cherry picked from commit d57f8a9) Co-authored-by: Damien <[email protected]>
…H-122568) (#122593) Co-authored-by: Damien <[email protected]>
…H-122568) (#122594) Co-authored-by: Damien <[email protected]>
Close PR for Azure Pipeline as it currently not supports |
According to latest update in Github Action, |
In current
ci.yml
, base OS image isubuntu-22.04
, should we move to the latest LTS verion:ubuntu-24.04
?To go further, if we should replace all current CI testing which run on
ubuntu-22.04
toubuntu-24.04
?Linked PRs
The text was updated successfully, but these errors were encountered: