ci: cover kilted and lyrical, drop the impossible rolling-main job - #111
Merged
Merged
Conversation
Rolling's `main` apt repo has no Resolute packages, so rolling+main cannot install and has been failing since at least 2026-07-13. Replace it with kilted and lyrical, matching moveit_visual_tools#157 and rviz_visual_tools#301. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
nbbrooks
force-pushed
the
nbbrooks/ci-matrix-distro-coverage
branch
from
August 12, 2026 04:37
4adbe33 to
c7668df
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Aligns distro coverage with moveit/moveit_visual_tools#157 and PickNikRobotics/rviz_visual_tools#301.
Jobs: 4 → 5 —
humble-main,jazzy-main,kilted-main,lyrical-main,rolling-testing.Why drop
rolling+mainindustrial_ci resolves
rollingtoubuntu:resolute, and Rolling'smainapt repo has no Resolute packages — they are all inros2-testingfor now. So the job cannot install and fails during setup:It had been red since at least 2026-07-13, independent of any code change. That masked real signal:
rolling-testingcorrectly went red on 2026-08-03 catching thetf2_ros0.46.2 breakage fixed in #110, but a half-red Rolling column looked normal, and 2.0.7-1 shipped the same day.What is added
kilted-main— released, previously untested here.lyrical-main— Ubuntu Resolute with a populatedmainrepo, so Resolute coverage no longer depends on Rolling's churn.OS_CODE_NAME: resoluteis pinned on both Resolute jobs. It matches industrial_ci's current default; pinning keeps it readable and correct if that default moves.Both Resolute jobs are blocking, unlike moveit_visual_tools#157 — that repo needs
NONBLOCKINGbecausemoveit_corehas no Resolute deb yet, whereaswarehouse_rosis upstream ofmoveit_coreand depends only on core ROS packages that are available on Resolute for both distros.continue-on-erroris still wired to aNONBLOCKINGkey so parking a job later is a one-line change.Merge note
Branch protection needs its required-check list updated:
rolling-mainis gone,kilted-mainandlyrical-mainare new. Required checks match by name, so a stale list will block merges on a job that no longer runs.