Skip to content

Change RHEL default to RHEL 10#870

Closed
christophebedard wants to merge 1 commit into
masterfrom
christophebedard/default-rhel-10
Closed

Change RHEL default to RHEL 10#870
christophebedard wants to merge 1 commit into
masterfrom
christophebedard/default-rhel-10

Conversation

@christophebedard
Copy link
Copy Markdown
Member

Description

In preparation for Lyrical.

Is this user-facing behavior change?

Did you use Generative AI?

Additional Information

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@cottsay
Copy link
Copy Markdown
Member

cottsay commented Apr 20, 2026

This should really be done in the same commit as Resolute, right?

@christophebedard
Copy link
Copy Markdown
Member Author

I don't really know. We kind of need to bump to RHEL 10 to merge #868. Bumping to Resolute at the same time makes sense, but it feels like such the wrong time to do it, when we're trying to get other things in.

@cottsay
Copy link
Copy Markdown
Member

cottsay commented Apr 20, 2026

We kind of need to bump to RHEL 10 to merge #868.

It looks like python3-pytest-asyncio is available for RHEL 9 via EPEL. Am I missing something? Why does moving to RHEL 10 block that PR?

@christophebedard
Copy link
Copy Markdown
Member Author

christophebedard commented Apr 20, 2026

I believe it requires Python >=3.12 (for some language features?). RHEL 9 has 3.9.

@cottsay
Copy link
Copy Markdown
Member

cottsay commented Apr 20, 2026

I believe it requires Python >=3.12 (for some language features?). RHEL 9 has 3.9.

ros2/rclpy#1620 requires Python >= 3.12? I'm confused.

@christophebedard
Copy link
Copy Markdown
Member Author

christophebedard commented Apr 20, 2026

it is using asyncio task groups, which were added in 3.11: https://docs.python.org/3/library/asyncio-task.html#task-groups. And apparently it's also using a Python 3.12 feature.

However, we are currently discussing having a check to disable the feature (or just the tests) if the Python version is too old: #Client Libraries Working Group > AsyncNode @ 💬.

@christophebedard
Copy link
Copy Markdown
Member Author

OK, as per #Client Libraries Working Group > AsyncNode @ 💬, Nadav is going to try skipping the tests so that we can stay on RHEL 9 for now. Closing this!

@christophebedard christophebedard deleted the christophebedard/default-rhel-10 branch April 20, 2026 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants