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

[CI] Fail in case of process crash #42

Merged
merged 2 commits into from
Mar 22, 2021

Conversation

3wnbr1
Copy link
Member

@3wnbr1 3wnbr1 commented Mar 22, 2021

This doesn't change the behavior of the launch system on the robots. The CI makes use of the titan package.
In titan launch files, it adds an event handler to catch any processes exiting and proceeds to a launcher shutdown, exiting with a non zero code

Fixes #40

Copy link
Member

@PhileasL PhileasL left a comment

Choose a reason for hiding this comment

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

It could be nice to write a documentation on the purpose of your work and how you achieve it

@3wnbr1 3wnbr1 merged commit 56e3e8a into master Mar 22, 2021
@3wnbr1 3wnbr1 deleted the 3wnbr1/fail-ci-in-case-of-process-crash branch March 22, 2021 11:42
3wnbr1 added a commit that referenced this pull request Mar 22, 2021
3wnbr1 added a commit that referenced this pull request Mar 24, 2021
mAxYoLo01 added a commit that referenced this pull request Jul 3, 2021
* Hello World!
Start of reformating of cetautomatix and strategix

* Blacked + continuation of robot.py reformating

* Strategix & Cetautomatix reformating finished! 🎉🎉🎉
Available actions: both Phares & Pavillon

* Update ros_ws.repos (#38)

* Build libceres-dev as N/A on Debian buster aarch64 (#41)

* Add ceres-solver
It's not available in debian buster aarch64

* [ros_ws] dependencies - override disable SSE for aarch64 cross build

* Fix foxy bondcpp version - 2.0.0

* Changed odom_callback to use kdl instead of tf2 ros

* Added tf2_kdl as dependency of cetautomatix

* [CI] Fail in case of process crash ⚠️ (#42)

* Added Gobelet support
Continued translating old action positions to new object classes
Few fixes & comments

* Reimplemented strategy modes

* Utils time 🎉 Plot map of all actions and plot of strategy modes

* Fix conflicts

* 🍺 Hi Ewen! 🍺

* 🍺 Blacked 🍺

Co-authored-by: Philéas LAMBERT <phileas.lambert@gmail.com>
Co-authored-by: Ewen BRUN <ewen.brun@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI can succeed even if a node crashes
2 participants