Install liblttng-ctl-dev instead of python3-lttng in Rolling

[christophebedard]

Required for ros2/ros2_tracing#81

We need liblttng-ctl-dev and can drop python3-lttng in Rolling. We still need python3-lttng in Iron, though.

Note that RHEL has liblttng-ctl version 2.12.11, while Ubuntu has 2.13.x.

[cottsay]

The corresponding package on RHEL9 is lttng-tools-devel - please update Dockerfile-RHEL appropriately.

[christophebedard]

I updated Dockerfile-RHEL. We should also probably test a RHEL CI build with the same settings as ros2/ros2_tracing#81 (comment) (I can't push the branch to this repo, though)

[christophebedard]

@cottsay can you force-push my PR branch to this repo so that I can test the RHEL CI job? Thank you!

[mjcarroll]

@cottsay can you force-push my PR branch to this repo so that I can test the RHEL CI job?

He's OOO, I pushed it.

[christophebedard]

Looks like I can't re-build the Docker image with RHEL/EL 9.2 due to a workaround for #727. Using 9.1 results in the failure described in #727: https://ci.ros2.org/job/ci_linux-rhel/517/. I guess we'll have to wait for this to be fixed.

[claraberendsen]

@christophebedard yes momentarily the 9.2 image is pinned to a stale one with the working dependencies for pyside2

• Use stale container image with AlmaLinux 9.2 and working dependencies #728

and all other options of building on RHEL 9 are going to result on failures due to the issue linked above.
For now sadly until upstream EPEL dependencies breaks are rebuilt for 9.3 and we can move the ci to use the new AlmaLinux 9.3 there is no easy way to change a dependency.

[clalancette]

We're still not quite to the place where we can take this; we first need to revert the 9.2-pinned PR (#728) and ensure everything works. Before we do that, we need to merge in ros-visualization/python_qt_binding#130 . But we are getting closer :).

[christophebedard]

Thanks for the update!

Looks like #728 has been reverted (#732) and ros-visualization/python_qt_binding#130 has been merged. Are we okay to proceed?

[claraberendsen]

@christophebedard Yes, Chris was able to unblock the RHEL situation. You should be unblocked here to run ci again to test your changes.

[christophebedard]

Thanks! Would you (or anyone) mind force-pushing my branch to this repo so that I can (re)run a RHEL job?

[claraberendsen]

Thanks! Would you (or anyone) mind force-pushing my branch to this repo so that I can (re)run a RHEL job?

Latest changes force pushed on christophebedard/install-liblttng-ctl-dev.

[christophebedard]

Looking good now, see new CI (including RHEL) at ros2/ros2_tracing#81 (comment). This is ready for a final review.

[christophebedard]

This should be ready to be merged.

[clalancette]

@christophebedard Am I correct in thinking that this needs to be merged in concert with ros2/ros2_tracing#81 , so that Rolling nightlies continue to build?

[christophebedard]

Indeed! I can't merge this PR but I can merge the ros2_tracing one right after.

[christophebedard]

@clalancette could you merge this PR? Then right after I can merge ros2/ros2_tracing#81.

[clalancette]

@clalancette could you merge this PR? Then right after I can merge ros2/ros2_tracing#81.

I'm just going to go ahead and merge in both.