Skip to content

Run the jazzy branch CI on jazzy - #157

Merged
aaronchongth merged 1 commit into
open-rmf:jazzyfrom
HappySamuel:ci/jazzy-dist-matrix
Nov 11, 2026
Merged

aaronchongth merged 1 commit into
open-rmf:jazzyfrom
HappySamuel:ci/jazzy-dist-matrix

Conversation

@HappySamuel

Copy link
Copy Markdown
Contributor

Bug fix

Fixed bug

All CI jobs on the jazzy branch fail before rmf_task is compiled, for example on #155:

rmf_task: No definition of [ament_cmake_uncrustify] for OS version [noble]
...
Could not find a package configuration file provided by "eigen3_cmake_module"

The workflows on this branch still build against rolling on noble. Rolling has moved to resolute, so rosdep can no longer resolve the dependencies on noble.

Fix applied

Set dist-matrix to jazzy / noble in build.yaml, asan.yaml and tsan.yaml, like the jazzy branch of rmf_ros2. Nothing else changes.

Testing

This PR's own CI runs with the new matrix. Locally on Ubuntu 24.04 / Jazzy, rmf_task and rmf_task_sequence from the jazzy branch (with #155) build and all their tests pass.

GenAI Use

We follow OSRA's policy on GenAI tools

  • I used a GenAI tool in this PR.
  • I did not use GenAI

Generated-by: Claude Code (Claude Opus 5.5) was used to find the cause of the CI failure, make the change and draft this description. I reviewed the change.

🤖 Generated with Claude Code

The workflows on this branch still built against rolling on noble. Rolling has
moved to resolute, so rosdep can no longer resolve the dependencies on noble
(eigen3_cmake_module, ament_cmake_uncrustify, ...) and every job fails before
compiling rmf_task. Build against jazzy instead, like rmf_ros2's jazzy branch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: HappySamuel <18137963+HappySamuel@users.noreply.github.com>
@mxgrey mxgrey added this to PMC Board Nov 11, 2026
@github-project-automation github-project-automation Bot moved this to Inbox in PMC Board Nov 11, 2026
@HappySamuel HappySamuel mentioned this pull request Nov 11, 2026
1 of 2 tasks

@aaronchongth aaronchongth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for fixing that, LGTM

@aaronchongth
aaronchongth merged commit 7613284 into open-rmf:jazzy Nov 11, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from Inbox to Done in PMC Board Nov 11, 2026
@HappySamuel
HappySamuel deleted the ci/jazzy-dist-matrix branch November 11, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants