-
Notifications
You must be signed in to change notification settings - Fork 483
Improve exit handling of O2HitMerger #14396
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
Improve exit handling of O2HitMerger #14396
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
Error while checking build/O2/fullCI_slc9 for 0a97792 at 2025-06-12 14:01: Full log here. |
|
@ktf: Clang-tidy reports errors and let's the FullCI fail. Could it be related to the recent code modifications in that part? |
23fa86b to
46a932c
Compare
So far, the exit status of O2HitMerger was not analysed. This could lead to situations where O2HitMerger was killed by the OS due to out-of-memory, yet the o2-sim simulator still exited as "successfull". This commit improves the handling. Problems in O2HitMerger will lead to exit code 1 of o2-sim.
46a932c to
b8f492f
Compare
So far, the exit status of O2HitMerger was not analysed. This could lead to situations where O2HitMerger was killed by the OS due to out-of-memory, yet the o2-sim simulator still exited as "successfull".
This commit improves the handling. Problems in O2HitMerger will lead to exit code 1 of o2-sim.