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.

Sign up for GitHub

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

Jump to bottom

🧑‍🌾 TestROS2TopicUseSimTime.test_pub_times failing in Linux Coverage #974

Open
Crola1702 opened this issue Sep 3, 2024 · 2 comments · May be fixed by #975
Open

🧑‍🌾 TestROS2TopicUseSimTime.test_pub_times failing in Linux Coverage #974

Crola1702 opened this issue Sep 3, 2024 · 2 comments · May be fixed by #975

Comments

@Crola1702
Copy link
Contributor

Bug report

Required Info:

  • Operating System:
    • Ubuntu Noble
  • Installation type:
    • Source
  • Version or commit hash:
    • Rolling

Steps to reproduce issue

  1. Run a build in nightly_linux_coverage
  2. See test ros2topic.ros2topic.test.test_use_sim_time.test_use_sim_time fail

Additional information

Reference build:

Test regression:

Log output:

FAIL: test_use_sim_time.TestROS2TopicUseSimTime.test_pub_times
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins-agent/workspace/nightly_linux_coverage/ws/install/launch_testing/lib/python3.12/site-packages/launch_testing/markers.py", line 61, in _wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jenkins-agent/workspace/nightly_linux_coverage/ws/src/ros2/ros2cli/ros2topic/test/test_use_sim_time.py", line 124, in test_pub_times
    assert command.wait_for_shutdown(timeout=5)
AssertionError: assert False
 +  where False = <bound method ProcessProxy.wait_for_shutdown of <launch_testing.tools.process.ProcessProxy object at 0x7f2122599580>>(timeout=5)
 +    where <bound method ProcessProxy.wait_for_shutdown of <launch_testing.tools.process.ProcessProxy object at 0x7f2122599580>> = <launch_testing.tools.process.ProcessProxy object at 0x7f2122599580>.wait_for_shutdown
@Crola1702 Crola1702 mentioned this issue Sep 3, 2024
Closed
@Crola1702 Crola1702 changed the title 🧑‍🌾 🧑‍🌾 TestROS2TopicUseSimTime.test_pub_times failing in Linux Coverage Sep 3, 2024
@fujitatomoya
Copy link
Collaborator

I cannot reproduce this issue with local environment. @Crola1702 how often this problem has been observed?

@fujitatomoya fujitatomoya transferred this issue from ros2/launch Feb 11, 2025
@fujitatomoya
Copy link
Collaborator

@Crola1702 the test failure comes from ros2cli, so i just transferred the issue here.

@fujitatomoya fujitatomoya linked a pull request Feb 11, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

start the simulation from 1 second for the test. ros2/ros2cli
2 participants
@fujitatomoya @Crola1702