feat(ci): test with CycloneDDS - #6
Merged
Merged
Conversation
CI inherited whatever RMW each distro defaults to (Fast DDS), while Duatic deploys Cyclone DDS. Every green build therefore certified a middleware nobody ships. Pin rmw_cyclonedds_cpp across the matrix so the tested configuration matches the deployed one. The gap surfaced as a flake: the kilted leg of duatic_duarover failed 10 of 20 identical nightly runs on the same main commit, always in the same Nav2 sim test, always burning the full waypoint timeout after a service or action response was dropped. Only the failing runs logged Fast DDS's "failed to send response ... rmw_response.cpp", and only kilted, which is on the Fast DDS 3.x line, was affected. Jazzy, on 2.x, was 6/6 green. Two things have to line up, since naming an RMW that is not installed fails at node startup rather than falling back. The deb is installed via ADDITIONAL_DEBS as ros-<distro>-rmw-cyclonedds-cpp, and the selection rides in on DOCKER_RUN_OPTS: industrial_ci builds inside Docker and only forwards the variables in its docker.env, which does not list RMW_IMPLEMENTATION, so a job-level env would never reach the tests. Both are hardcoded side by side in the industrial_ci step rather than exposed as an input, since no repo needs a different middleware.
mbloechli
force-pushed
the
feat/explicit-rmw
branch
from
August 25, 2026 13:45
6ce58ff to
2765825
Compare
mbloechli
marked this pull request as ready for review
August 25, 2026 14:55
mbloechli
changed the title
mbloechli
changed the title
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
1 participant
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.
use middleware cyclonedds