Skip to content

gh-122544: Change OS image in GitHub Actions to ubuntu-24.04 #122566

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

Merged
merged 4 commits into from
Nov 5, 2024

Conversation

Damien-Chen
Copy link
Contributor

@Damien-Chen Damien-Chen commented Aug 1, 2024

Change base OS image in Github Actions from ubuntu-22.04 to ubuntu-24.04

@bedevere-app
Copy link

bedevere-app bot commented Aug 1, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@hugovk hugovk changed the title gh-122544: Change OS image in github workflow to ubuntu-24.04 gh-122544: Change OS image in GitHub Actions to ubuntu-24.04 Aug 2, 2024
@AA-Turner
Copy link
Member

Failing due to Unable to locate package libmpdec-dev, as the libmpdec-dev package does not appear in Ubuntu 24.04 LTS. According to this comment, the removal is intentional as it is no longer needed, see the debian mailing list post.

@Damien-Chen are you able to test this please?

A

@Damien-Chen
Copy link
Contributor Author

Failing due to Unable to locate package libmpdec-dev, as the libmpdec-dev package does not appear in Ubuntu 24.04 LTS. According to this comment, the removal is intentional as it is no longer needed, see the debian mailing list post.

@Damien-Chen are you able to test this please?

A

Sure!. No problem!

@AA-Turner
Copy link
Member

The regen-all check is failing, I assume something changed between LTS versions -- can you run make regen-all on 24.04 and push that?

A

@Damien-Chen
Copy link
Contributor Author

The regen-all check is failing, I assume something changed between LTS versions -- can you run make regen-all on 24.04 and push that?

A

Let me check if I can test it locally on my WSL2 ubuntu-24.04 or use another local machine to install ubuntu-24.04. Or if I can trigger all pipline on my fork repo so that I I don’t have to confirm the result every time I commit to this PR.

@Damien-Chen
Copy link
Contributor Author

Damien-Chen commented Aug 3, 2024

The regen-all check is failing, I assume something changed between LTS versions -- can you run make regen-all on 24.04 and push that?

A

It means that adding make regen-all before git add -u like the below image show?

  - name: Check for changes
    run: |
      make regen-all
      git add -u
      changes=$(git status --porcelain)

@Damien-Chen
Copy link
Contributor Author

It seems same error, I run on my forked repo in this

@AA-Turner
Copy link
Member

Sorry, I meant you need to run make regen-all locally and commit the output of that to your branch.

@Damien-Chen
Copy link
Contributor Author

Sorry, I meant you need to run make regen-all locally and commit the output of that to your branch.

It seems that the below three file cause error in ubuntu-24.04
aclocal.m4, config.guess, configure.

I test it in several method. First is run make regen-all on my local ubuntun-24.04 machine, and convert the whole output into a script, and then execute it before running Check for changes test but it fails.
Then I copy the above three mention file generated in ubuntu-24.04, and overwrite the original three file that in ubuntu-22.04 and it seems pass the check.

Maybe there are some modification between ubuntu-22.04 and ubuntu-24.04 cause the above mentioned three file different ?

The whole process and result were conduct in my fork repo here

ubuntu-24.04output.txt

And the attached file is the output of running make regen-all in ubuntu-24.04.

@Damien-Chen
Copy link
Contributor Author

For ubuntu-24.04, these three file aclocal.m4, config.guess, configure are different than ubuntu-22.04. Maybe we should update these file together.

@hugovk
Copy link
Member

hugovk commented Sep 23, 2024

(Updating branch to re-trigger CLA bot, which got stuck a few days ago.)

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, CI changes look good to me. I'd like someone else (@corona10? @erlend-aasland?) to review the configure + libmpdec-dev changes.

@hugovk
Copy link
Member

hugovk commented Oct 10, 2024

@Damien-Chen Please could you resolve the conflicts?

@erlend-aasland @corona10 Please could you check the configure + libmpdec-dev changes here?

We'll also need to upgrade doctest soon (see #125236), but that can be a separate PR.

@erlend-aasland
Copy link
Contributor

I'll check it later today, when I'm back on my laptop.

@Damien-Chen
Copy link
Contributor Author

Resolved conflict.

@Damien-Chen
Copy link
Contributor Author

Should we merge this as early as possible?

@AA-Turner
Copy link
Member

There's no rush to merge this, it is currently blocked on review of the autoconf / configure changes. Most likely we will need to co-ordinate updating e.g. the docker images to Ubuntu 24 too.

cc @corona10

A

ambv added a commit that referenced this pull request Nov 22, 2024
@Damien-Chen Damien-Chen deleted the build-ubuntu-24.04 branch November 24, 2024 03:36
hugovk added a commit that referenced this pull request Jan 11, 2025
@hugovk
Copy link
Member

hugovk commented Feb 18, 2025

Let's backport this to 3.9-3.11, they're still using some ubuntu-20.04 which is deprecated and will be fully unsupported by 2025-04-01, with brownouts in March:

actions/runner-images#11101

@miss-islington-app
Copy link

Thanks @Damien-Chen for the PR, and @erlend-aasland for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @Damien-Chen for the PR, and @erlend-aasland for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @Damien-Chen for the PR, and @erlend-aasland for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Sorry, @Damien-Chen and @erlend-aasland, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker fc233f46d3761b4e808be2c44fda0b843179004e 3.9

@miss-islington-app
Copy link

Sorry, @Damien-Chen and @erlend-aasland, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker fc233f46d3761b4e808be2c44fda0b843179004e 3.10

@miss-islington-app
Copy link

Sorry, @Damien-Chen and @erlend-aasland, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker fc233f46d3761b4e808be2c44fda0b843179004e 3.11

hugovk pushed a commit to hugovk/cpython that referenced this pull request Feb 18, 2025
@bedevere-app
Copy link

bedevere-app bot commented Feb 18, 2025

GH-130260 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Feb 18, 2025
@bedevere-app
Copy link

bedevere-app bot commented Feb 18, 2025

GH-130268 is a backport of this pull request to the 3.10 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.10 only security fixes label Feb 18, 2025
hugovk pushed a commit to hugovk/cpython that referenced this pull request Feb 19, 2025
@bedevere-app
Copy link

bedevere-app bot commented Feb 19, 2025

GH-130295 is a backport of this pull request to the 3.9 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.9 only security fixes label Feb 19, 2025
@bedevere-app
Copy link

bedevere-app bot commented Feb 19, 2025

GH-130260 is a backport of this pull request to the 3.11 branch.

@bedevere-app
Copy link

bedevere-app bot commented Feb 19, 2025

GH-130268 is a backport of this pull request to the 3.10 branch.

@bedevere-app
Copy link

bedevere-app bot commented Feb 19, 2025

GH-130295 is a backport of this pull request to the 3.9 branch.

ambv pushed a commit that referenced this pull request Feb 19, 2025
…H-122566) (#130268)

Co-authored-by: Damien <[email protected]>
Co-authored-by: Serhiy Storchaka <[email protected]>
Co-authored-by: James De Bias <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change base OS image to Ubuntu-24.04 in CI testing
5 participants