[21245] Fix topic interference on `liveliness_changed` status #4988

MiguelCompany · 2024-06-24T13:04:46Z

Description

PR #4822 (and its backports) introduced a regression that was not detected until we requested ROS 2 rolling to use branch 2.14.x here

This PR:

Adds infrastructure to create Blackbox tests that simulate the behavior of ROS 2 node
Adds a ROS2 black-box test trying to mimick the test that regressed
Add a standard black-box regression test for the issue discovered
Fixes the issue

@Mergifyio backport 2.14.x 2.13.x 2.10.x 2.6.x

Contributor Checklist

Commit messages follow the project guidelines.
The code follows the style guidelines of this project.
Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
Any new/modified methods have been properly documented using Doxygen.
N/A Any new configuration API has an equivalent XML API (with the corresponding XSD extension)
Changes are backport compatible: they do NOT break ABI nor change library core behavior.
Changes are API compatible.
N/A New feature has been added to the versions.md file (if applicable).
N/A New feature has been documented/Current behavior is correctly described in the documentation.
Applicable backports have been included in the description.

Reviewer Checklist

The PR has a milestone assigned.
The title and description correctly express the PR's purpose.
Check contributor checklist is correct.
Check CI results: changes do not issue any warning.
Check CI results: failing tests are unrelated with the changes.

test/blackbox/common/BlackboxTestsLivelinessQos.cpp

test/blackbox/common/ros2/Node.hpp

EduPonz

LGTM with green CI

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

MiguelCompany · 2024-07-04T12:28:18Z

@Mergifyio backport 2.14.x

mergify · 2024-07-04T12:28:40Z

backport 2.14.x

🛑 Command `backport 2.14.x` cancelled because of a new `backport` command with different arguments

MiguelCompany · 2024-07-04T14:18:37Z

@Mergifyio backport 2.14.x

mergify · 2024-07-04T14:18:46Z

backport 2.14.x

✅ Backports have been created

#5032 [21245] Fix topic interference on liveliness_changed status (backport #4988) has been created for branch 2.14.x but encountered conflicts

* Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp

* Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com>

* Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934)

* Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934) # Conflicts: # src/cpp/rtps/reader/StatelessReader.cpp

* Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp

…#5044) * Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934) Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934) # Conflicts: # src/cpp/rtps/reader/StatelessReader.cpp

…#5045) * Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) # Conflicts: # src/cpp/rtps/reader/StatefulReader.cpp # src/cpp/rtps/reader/StatelessReader.cpp # test/blackbox/common/BlackboxTestsLivelinessQos.cpp * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934) # Conflicts: # src/cpp/rtps/reader/StatelessReader.cpp Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Fix topic interference on `liveliness_changed` status (#4988) * Refs #21189. Basic infrastructure for ROS2 blackbox tests. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added callback for liveliness_changed event. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added ROS2 regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Added blackbox regression test. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatefulReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix StatelessReader. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Uncrustify and doxygen. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Change liveliness announcement periods. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Create ROS 2 builtin endpoints. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix type and QoS on ros_discovery_info. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Avoid collision with true ROS 2 topics. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Uncrustify. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix warnings on MacOS. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21245. Fix tsan reported deadlock. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix long-standing deadlock in WLP. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #21189. Fix build after rebase. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 9243ead) * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Fix build after backport. Signed-off-by: Miguel Company <miguelcompany@eprosima.com> --------- Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 4a6b934)

MiguelCompany added this to the v3.0.0 milestone Jun 24, 2024

elianalf added the needs-review PR that is ready to be reviewed label Jun 24, 2024

EduPonz suggested changes Jun 25, 2024

View reviewed changes

test/blackbox/common/BlackboxTestsLivelinessQos.cpp Outdated Show resolved Hide resolved

test/blackbox/common/ros2/Node.hpp Outdated Show resolved Hide resolved

MiguelCompany force-pushed the bugfix/21189/master branch from a875bb7 to 30f69e5 Compare June 25, 2024 10:35

MiguelCompany requested a review from EduPonz June 25, 2024 10:36

github-actions bot added the ci-pending PR which CI is running label Jun 25, 2024

EduPonz requested review from EduPonz and removed request for EduPonz June 25, 2024 12:14

EduPonz previously approved these changes Jun 25, 2024

View reviewed changes

MiguelCompany dismissed EduPonz’s stale review via 3c11718 June 25, 2024 12:20

MiguelCompany requested a review from EduPonz June 25, 2024 12:20

EduPonz previously approved these changes Jun 25, 2024

View reviewed changes

MiguelCompany dismissed EduPonz’s stale review via b137c03 June 25, 2024 15:04

MiguelCompany force-pushed the bugfix/21189/master branch from 3c11718 to b137c03 Compare June 25, 2024 15:04

MiguelCompany requested review from EduPonz and removed request for EduPonz June 25, 2024 15:04

EduPonz previously approved these changes Jun 26, 2024

View reviewed changes

elianalf removed the needs-review PR that is ready to be reviewed label Jul 1, 2024

EduPonz added the to-do label Jul 1, 2024

MiguelCompany added 10 commits July 3, 2024 16:39

Refs #21189. Basic infrastructure for ROS2 blackbox tests.

2062910

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21189. Added callback for liveliness_changed event.

36cc57e

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21189. Added ROS2 regression test.

4fc8cd4

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21189. Added blackbox regression test.

7d813bf

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21189. Fix StatefulReader.

05e930a

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21189. Fix StatelessReader.

057730c

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21189. Uncrustify and doxygen.

1d754b8

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21245. Change liveliness announcement periods.

33cbfe5

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21245. Create ROS 2 builtin endpoints.

b9fd36f

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Refs #21245. Fix type and QoS on ros_discovery_info.

5c42e28

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

MiguelCompany force-pushed the bugfix/21189/master branch from a6444d7 to 95865e4 Compare July 3, 2024 14:40

MiguelCompany added needs-review PR that is ready to be reviewed and removed to-do ci-pending PR which CI is running labels Jul 3, 2024

MiguelCompany requested a review from EduPonz July 3, 2024 14:40

github-actions bot added the ci-pending PR which CI is running label Jul 3, 2024

Refs #21189. Fix build after rebase.

78e0a33

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

MiguelCompany requested review from EduPonz and removed request for EduPonz July 4, 2024 05:55

EduPonz approved these changes Jul 4, 2024

View reviewed changes

MiguelCompany merged commit 9243ead into master Jul 4, 2024
16 of 17 checks passed

MiguelCompany deleted the bugfix/21189/master branch July 4, 2024 12:28

mergify bot mentioned this pull request Jul 4, 2024

[21245] Fix topic interference on liveliness_changed status (backport #4988) #5032

Merged

12 tasks

This was referenced Jul 9, 2024

[21245] Fix topic interference on liveliness_changed status (backport #4988) (backport #5032) #5044

Merged

[21245] Fix topic interference on liveliness_changed status (backport #4988) (backport #5032) #5045

Merged

mergify bot mentioned this pull request Jul 9, 2024

[21245] Fix topic interference on liveliness_changed status (backport #4988) (backport #5032) #5046

Open

12 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[21245] Fix topic interference on `liveliness_changed` status #4988

[21245] Fix topic interference on `liveliness_changed` status #4988

MiguelCompany commented Jun 24, 2024 •

edited by EduPonz

Loading

EduPonz left a comment

MiguelCompany commented Jul 4, 2024

mergify bot commented Jul 4, 2024 •

edited

Loading

MiguelCompany commented Jul 4, 2024

mergify bot commented Jul 4, 2024 •

edited

Loading

[21245] Fix topic interference on liveliness_changed status #4988

[21245] Fix topic interference on liveliness_changed status #4988

Conversation

MiguelCompany commented Jun 24, 2024 • edited by EduPonz Loading

Description

Contributor Checklist

Reviewer Checklist

EduPonz left a comment

Choose a reason for hiding this comment

MiguelCompany commented Jul 4, 2024

mergify bot commented Jul 4, 2024 • edited Loading

🛑 Command backport 2.14.x cancelled because of a new backport command with different arguments

MiguelCompany commented Jul 4, 2024

mergify bot commented Jul 4, 2024 • edited Loading

✅ Backports have been created

[21245] Fix topic interference on `liveliness_changed` status #4988

[21245] Fix topic interference on `liveliness_changed` status #4988

MiguelCompany commented Jun 24, 2024 •

edited by EduPonz

Loading

mergify bot commented Jul 4, 2024 •

edited

Loading

🛑 Command `backport 2.14.x` cancelled because of a new `backport` command with different arguments

mergify bot commented Jul 4, 2024 •

edited

Loading