-
Notifications
You must be signed in to change notification settings - Fork 43
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
Add Ubuntu 24.04 Noble and switch Rolling to it #658
Add Ubuntu 24.04 Noble and switch Rolling to it #658
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #658 +/- ##
==========================================
- Coverage 92.47% 91.09% -1.38%
==========================================
Files 8 8
Lines 186 191 +5
Branches 22 24 +2
==========================================
+ Hits 172 174 +2
- Misses 14 17 +3 ☔ View full report in Codecov by Sentry. |
c991657
to
46e3cbe
Compare
733a881
to
0deb0df
Compare
The |
Signed-off-by: Christian Henkel <christian.henkel2@de.bosch.com>
I can confirm. It works for me: https://github.com/ros/diagnostics/actions/runs/8439160454/job/23113100580?pr=335 |
Looks like there is an incompatibility between
See this CI job from the The |
You're not hitting the issue I mentioned in my comment above? Ah, probably because the package you're testing doesn't use |
Yes, I can also confirm the flake8 issue. |
The downgrade to
|
5b12f23
to
04cf11b
Compare
It may be because Can you try to do ros2/ci#765 instead of using |
Well I did it and it seems to work! |
Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
We can rely on rosdep for: 1. Some flake8 plugins, see: ros2/ros2_documentation#3945 2. libtinyxml2-dev from tinyxml2_vendor 3. libasio-dev for Fast DDS Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
1a3d4c1
to
0876b68
Compare
Since this works, I'll go ahead and merge it. I may not create a new release just yet since the transition to Noble is not officially complete. I'll let people using |
Relates to ros-tooling/action-ros-ci#842
See also ros-tooling/setup-ros-docker#69 (required for the
Test on a setup-ros-docker container
CI job)Closes #659
Rolling is moving to Ubuntu 24.04 Noble: https://discourse.ros.org/t/preparing-ros-2-rolling-for-the-transition-to-ubuntu-24-04/35673.
Note: the
ubuntu-24.04
GitHub runner is not available yet: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#choosing-github-hosted-runners. I changed the workflow configs to use a Ubuntu Noble Docker image on top ofubuntu-latest
. We could just keep this and not have to remove the workaround + bring it back for Ubuntu 26.04.I also removed some
apt
from the list for Noble, since we can just rely onrosdep
:libtinyxml2-dev
installed fortinyxml2_vendor