Skip to content
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

Liveliness demos with Fast-RTPS don't seem to work #351

Closed
clalancette opened this issue May 23, 2019 · 2 comments · Fixed by ros2/rmw_fastrtps#282
Closed

Liveliness demos with Fast-RTPS don't seem to work #351

clalancette opened this issue May 23, 2019 · 2 comments · Fixed by ros2/rmw_fastrtps#282
Labels
bug Something isn't working

Comments

@clalancette
Copy link
Contributor

Bug report

I'm following the Quality of Service demos at https://github.com/ros2/demos/blob/master/quality_of_service_demo/README.md , and none of the liveliness demos seem to work for me.

Required Info:

  • Operating System:
    • Ubuntu 18.04
  • Installation type:
    • Source
  • Version or commit hash:
  • DDS implementation:
    • Fast-RTPS
  • Client library (if applicable):
    • rclcpp/rclpy

Steps to reproduce issue

ros2 run quality_of_service_demo_cpp liveliness 1000 --kill-publisher-after 2000

Expected behavior

After 2 seconds, the publisher is killed and the subscriber gets a callback which prints out some information about liveliness.

Actual behavior

After 2 seconds, the publisher seems to be killed (at least, it stops printing), but we get no information about liveliness.

Additional information

This is failing for both quality_of_service_demo_cpp and quality_of_service_demo_py. Further, trying to run ros2 run quality_of_service_demo_cpp liveliness 250 --node-assert-period 0 --policy MANUAL_BY_NODE throws an exception on startup saying "Manual liveliness unsupported for fastrtps", but that is not mentioned in the README. @emersonknapp FYI.

@emersonknapp
Copy link
Contributor

I have opened this PR to disable all Liveliness for Fast-RTPS ros2/rmw_fastrtps#282 - we thought Liveliness would be supported in the 1.8 release but it ended up not being the case.

Do you think we should also update the demo README to mention this limitation?

@jacobperron
Copy link
Member

I would update the README since it might be perceived as a bug to users (#365).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants