Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Conversation

@Karsten1987
Copy link

eloquent backport of #1033

* make transient local reliable

Signed-off-by: Karsten Knese <[email protected]>

* fix master

Signed-off-by: Karsten Knese <[email protected]>

* add launch test

Signed-off-by: Karsten Knese <[email protected]>

* make it actual latched

Signed-off-by: Karsten Knese <[email protected]>

* alpha sort

Signed-off-by: Karsten Knese <[email protected]>

* add launch_test dependency

Signed-off-by: Karsten Knese <[email protected]>

* more dependencies

Signed-off-by: Karsten Knese <[email protected]>

* remove debug print

Signed-off-by: Karsten Knese <[email protected]>

* is_initialized -> ok

Signed-off-by: Karsten Knese <[email protected]>

* Update gazebo_ros/test/entity_spawner.test.py

Co-Authored-By: chapulina <[email protected]>

* use erase-remove idiom

Signed-off-by: Karsten Knese <[email protected]>

* use ReadyToTest()

Signed-off-by: Karsten Knese <[email protected]>

Co-authored-by: chapulina <[email protected]>
@chapulina chapulina added the ros2 label Jan 24, 2020
@Karsten1987
Copy link
Author

test failures seem unrelated to the fix and the newly introduced test ran successfully.

@Karsten1987 Karsten1987 requested a review from chapulina January 24, 2020 21:36
@chapulina
Copy link
Contributor

Actually it looks like the new test is responsible for the test failures. It looks like gzserver didn't shut down properly at the end of the test:

11:32:38 19: [INFO] [gzserver-2]: sending signal 'SIGINT' to process[gzserver-2]
11:32:38 19: [INFO] [mock_robot_state_publisher-1]: sending signal 'SIGINT' to process[mock_robot_state_publisher-1]
11:32:38 19: [mock_robot_state_publisher-1] [INFO] [mock_robot_state_publisher]: Publishing on /mock_robot_description
11:32:38 19: [mock_robot_state_publisher-1] [INFO] [rclcpp]: signal_handler(signal_value=2)
11:32:38 19: [INFO] [mock_robot_state_publisher-1]: process has finished cleanly [pid 7382]
11:32:43 19: [ERROR] [gzserver-2]: process[gzserver-2] failed to terminate '5' seconds after receiving 'SIGINT', escalating to 'SIGTERM'
11:32:43 19: [INFO] [gzserver-2]: sending signal 'SIGTERM' to process[gzserver-2]
11:32:43 19: [ERROR] [gzserver-2]: process has died [pid 7383, exit code -15, cmd 'gzserver                                                                    -s libgazebo_ros_init.so   -s libgazebo_ros_factory.so       '].

Which makes all subsequent tests fail with

11:32:46 7: �[1;31m[Err] [Master.cc:96] �[0m�[1;31mEXCEPTION: �[0m�[1;31mUnable to start server[bind: Address already in use]. There is probably another Gazebo process running.
11:32:46 7: �[0m�[1;31m
11:32:46 7: �[0m�[1;31m�[0mterminate called after throwing an instance of 'gazebo::common::Exception'

Could be flaky, let's try again.

@chapulina
Copy link
Contributor

@ros-pull-request-builder retest this please

@chapulina
Copy link
Contributor

chapulina commented Jan 24, 2020

Same thing again. I wonder if it's somehow related to the CI infrastructure, since #1033 tested ok on ci.ros2.org.

I'm adding PR CI for eloquent to the OSRF farm so we have another point of reference: https://bitbucket.org/osrf/release-tools/pull-requests/945/

@chapulina
Copy link
Contributor

@osrf-jenkins run tests please

@hidmic
Copy link

hidmic commented Jan 28, 2020

It looks like gzserver didn't shut down properly at the end of the test:

Maybe 5 seconds isn't enough for Gazebo to shut down? It'd be a bit unfortunate, but you can try bumping up SIGTERM and SIGKILL timeouts.

@chapulina
Copy link
Contributor

Thanks, @hidmic. A combination of the timeouts and #1037 worked locally for me: 36e9bd3

Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the endurance

@chapulina chapulina merged commit 63bec81 into ros-simulation:eloquent Feb 6, 2020
@Karsten1987 Karsten1987 deleted the backport_1033 branch February 10, 2020 18:39
Karsten1987 added a commit to Karsten1987/gazebo_ros_pkgs that referenced this pull request Feb 10, 2020
…os-simulation#1036)

* [ros2] make transient local reliable (ros-simulation#1033)

* make transient local reliable

Signed-off-by: Karsten Knese <[email protected]>

* fix master

Signed-off-by: Karsten Knese <[email protected]>

* add launch test

Signed-off-by: Karsten Knese <[email protected]>

* make it actual latched

Signed-off-by: Karsten Knese <[email protected]>

* alpha sort

Signed-off-by: Karsten Knese <[email protected]>

* add launch_test dependency

Signed-off-by: Karsten Knese <[email protected]>

* more dependencies

Signed-off-by: Karsten Knese <[email protected]>

* remove debug print

Signed-off-by: Karsten Knese <[email protected]>

* is_initialized -> ok

Signed-off-by: Karsten Knese <[email protected]>

* Update gazebo_ros/test/entity_spawner.test.py

Co-Authored-By: chapulina <[email protected]>

* use erase-remove idiom

Signed-off-by: Karsten Knese <[email protected]>

* use ReadyToTest()

Signed-off-by: Karsten Knese <[email protected]>

Co-authored-by: chapulina <[email protected]>

* Set timeout and call gzserver directly

Signed-off-by: Louise Poubel <[email protected]>

Co-authored-by: chapulina <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants