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

[Test failure] rclcpp/ test_publisher.intra_process_inter_process_mix_transient_local #366

Closed
Yadunund opened this issue Dec 28, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@Yadunund
Copy link
Member

Yadunund commented Dec 28, 2024

Following #365, #317 no longer fails but we have another failure in test_publisher.

62: [ RUN      ] TestPublisher.intra_process_inter_process_mix_transient_local
62: [WARN] [1735376330.176345205] [rclcpp]: Calling set_on_new_intra_process_message_callback for subscription with IPC disabled
62: /home/yadunund/ros2_rolling/src/ros2/rclcpp/rclcpp/test/rclcpp/test_publisher.cpp:822: Failure
62: Expected equality of these values:
62:   executor.spin_until_future_complete(inter_callback_future, std::chrono::milliseconds(100))
62:     Which is: TIMEOUT (2)
62:   rclcpp::FutureReturnCode::SUCCESS
62:     Which is: SUCCESS (0)
62: 
62: [  FAILED  ] TestPublisher.intra_process_inter_process_mix_transient_local (211 ms)
@Yadunund Yadunund added the bug Something isn't working label Dec 28, 2024
@Yadunund
Copy link
Member Author

This test was recently added to rclcpp: ros2/rclcpp#2708

@Yadunund
Copy link
Member Author

I'm unable to reproduce this error after restarting the zenoh router. The router had been running for a long time and over the course of running tests across across rcl, rcl_lifecycle, rcl_action, test_communication, test_quality_of_service, rclcpp, rclcpp_components, rclcpp_lifecycle, rclcpp_action, and rclpy multiple times.

It feels like the router might have got into a weird state and hence was not handing the discovery of pub caches/querying subscribers. Fyi @JEnoch, @YuanYuYuan.

Closing this out for now.

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

No branches or pull requests

1 participant