Skip to content

[ci] Use alma8 image with Python 3.12#19720

Closed
guitargeek wants to merge 1 commit intoroot-project:masterfrom
guitargeek:alma8-py312
Closed

[ci] Use alma8 image with Python 3.12#19720
guitargeek wants to merge 1 commit intoroot-project:masterfrom
guitargeek:alma8-py312

Conversation

@guitargeek
Copy link
Contributor

Let's see if this works. We need to update the Python version in alma8, because right now we're using a random Python version in Alma 8 that is not supported by RHEL anymore.

See https://access.redhat.com/support/policy/updates/rhel-app-streams-life-cycle

The only Python versions in RHEL 8 that are supported until its end-of-life are Python 3.6 and Python 3.12. Since ROOT requires Python 3.12, we need to use Alma 8 with Python 3.12.

Right now, our CI images install Python 3.8 on RHEL 3.8, but this intermediate version is not supported anymore by RHEL.

@guitargeek guitargeek self-assigned this Aug 21, 2025
@guitargeek guitargeek requested a review from dpiparo as a code owner August 21, 2025 22:23
@guitargeek guitargeek added clean build Ask CI to do non-incremental build on PR in:CI labels Aug 21, 2025
@github-actions
Copy link

github-actions bot commented Aug 22, 2025

Test Results

    22 files      22 suites   3d 16h 22m 52s ⏱️
 3 689 tests  3 684 ✅ 0 💤 5 ❌
79 229 runs  79 224 ✅ 0 💤 5 ❌

For more details on these failures, see this check.

Results for commit 2881e63.

♻️ This comment has been updated with latest results.

@hahnjo
Copy link
Member

hahnjo commented Aug 22, 2025

I think this is a bad idea: As far as I can tell, alma8 is currently the only platform testing our minimum of Python 3.8. In my opinion, if we claim to support it, we must make sure it works. Is there a concrete problem of keeping Python 3.8 for alma8? I don't think we care about updates and security fixes for our CI...

@guitargeek
Copy link
Contributor Author

I think this is a bad idea: As far as I can tell, alma8 is currently the only platform testing our minimum of Python 3.8. In my opinion, if we claim to support it, we must make sure it works.

Fortunately this is not a problem anymore, because we have nightly Python builds for all Python version we support.

Is there a concrete problem of keeping Python 3.8 for alma8? I don't think we care about updates and security fixes for our CI...

We use the same config for the release, and there we should care about it. Right now, we are shipping binaries built against a Python version that RHEL retired over 2 years ago:

Screenshot_2025-08-19_15-57-58

@hahnjo
Copy link
Member

hahnjo commented Aug 22, 2025

Fortunately this is not a problem anymore, because we have nightly Python builds for all Python version we support.

Yes, but not for every PR build that may break it by using syntax that doesn't work in Python 3.8...

@guitargeek guitargeek marked this pull request as draft August 22, 2025 09:51
@guitargeek
Copy link
Contributor Author

guitargeek commented Aug 22, 2025

That's true, so maybe we can make this change to the CI only once we change the minimum supported Python version.

In any case, the tests don't even pass 😆 Let's maybe come back to this later when we're actually getting closer to 6.38 branching/release.

@guitargeek guitargeek closed this Oct 19, 2025
@guitargeek guitargeek deleted the alma8-py312 branch October 19, 2025 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR in:CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants