-
Notifications
You must be signed in to change notification settings - Fork 240
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
Fix for ros2 bag play exit with non-zero code on SIGINT #1126
Fix for ros2 bag play exit with non-zero code on SIGINT #1126
Conversation
0f7b6db
to
b1de937
Compare
- Don't allow to propagate KeyboardInterrupt exception on upper level from python wrapper for play verb. - Fixed incorrect errors check in process_execution infrastructure when SIG_INT not processed in child process. Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
b1de937
to
476291a
Compare
Gist: https://gist.githubusercontent.com/MichaelOrlov/aade93f33f4aaa1e9c01f5c1b03a246d/raw/e8932793f1fbb3eb9403cca35e2faed867f0419b/ros2.repos |
@emersonknapp @jhdcs @gbiggs Kindly ping for review |
https://github.com/Mergifyio backport humble |
✅ Backports have been created
|
- Don't allow to propagate KeyboardInterrupt exception on upper level from python wrapper for play verb. - Fixed incorrect errors check in process_execution infrastructure when SIG_INT not processed in child process. Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Signed-off-by: Michael Orlov <michael.orlov@apex.ai> (cherry picked from commit 30d3889)
- Don't allow to propagate KeyboardInterrupt exception on upper level from python wrapper for play verb. - Fixed incorrect errors check in process_execution infrastructure when SIG_INT not processed in child process. Signed-off-by: Michael Orlov <michael.orlov@apex.ai> Signed-off-by: Michael Orlov <michael.orlov@apex.ai> (cherry picked from commit 30d3889) Co-authored-by: Michael Orlov <michael.orlov@apex.ai>
Fix for ros2 bag play exit with non-zero code on SIGINT
ros2 bag play
exiting with code 2 by SIGINT or CTRL+C #1123KeyboardInterrupt
exception on upper level from python wrapper forplay
verb.SIGINT
not processed in child process.