Skip to content

Conversation

@sawenzel
Copy link
Collaborator

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.

@sawenzel sawenzel requested a review from a team as a code owner June 12, 2025 09:41
@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@alibuild
Copy link
Collaborator

Error while checking build/O2/fullCI_slc9 for 0a97792 at 2025-06-12 14:01:

input queue has size 523
output queue has size 2
input queue has size 484
output queue has size 2
input queue has size 447
output queue has size 2
input queue has size 425
output queue has size 2
input queue has size 423
output queue has size 2
input queue has size 397
output queue has size 2
input queue has size 369
output queue has size 2
input queue has size 336
output queue has size 2
input queue has size 312
output queue has size 2
input queue has size 281
output queue has size 2
input queue has size 250
output queue has size 2
input queue has size 214
output queue has size 2
input queue has size 184
output queue has size 2
input queue has size 142
output queue has size 2
input queue has size 101
output queue has size 2
input queue has size 61
output queue has size 2
input queue has size 25
output queue has size 2
input queue has size 0
output queue has size 2
input queue has size 0
output queue has size 2
input queue has size 0
output queue has size 2
input queue has size 0
output queue has size 2
++ cp thinned_compile_commands.json compile_commands.json
++ CHECKS='-*,modernize-avoid-bind,modernize-deprecated-headers,modernize-make-shared,modernize-raw-string-literal,modernize-redundant-void-arg,modernize-replace-auto-ptr,modernize-replace-random-shuffle,modernize-shrink-to-fit,modernize-unary-static-assert,modernize-use-equals-default,modernize-use-noexcept,modernize-use-nullptr,modernize-use-override,modernize-use-transparent-functors,modernize-use-uncaught-exceptions,readability-braces-around-statements'
++ tee error-log.txt
+++ which O2codecheck
+++ find /sw/slc9_x86-64/GCC-Toolchain/v14.2.0-alice2-1/lib -name crtbegin.o -exec dirname '{}' ';'
++ run_O2CodeChecker.py -j 32 -clang-tidy-binary /sw/slc9_x86-64/o2codechecker/v18.1.2.1-local1/bin/O2codecheck -clang-apply-replacements-binary /sw/slc9_x86-64/Clang/v18.1.8-21/bin-safe/clang-apply-replacements -extra-args=--extra-arg=--gcc-install-dir=/sw/slc9_x86-64/GCC-Toolchain/v14.2.0-alice2-1/lib/gcc/x86_64-unknown-linux-gnu/14.2.0 '-header-filter=.*SOURCES(?!.*/3rdparty/).*' '-checks=-*,modernize-avoid-bind,modernize-deprecated-headers,modernize-make-shared,modernize-raw-string-literal,modernize-redundant-void-arg,modernize-replace-auto-ptr,modernize-replace-random-shuffle,modernize-shrink-to-fit,modernize-unary-static-assert,modernize-use-equals-default,modernize-use-noexcept,modernize-use-nullptr,modernize-use-override,modernize-use-transparent-functors,modernize-use-uncaught-exceptions,readability-braces-around-statements'
No checks enabled.
Unable to run clang-tidy.

Full log here.

@sawenzel
Copy link
Collaborator Author

@ktf: Clang-tidy reports errors and let's the FullCI fail. Could it be related to the recent code modifications in that part?

@sawenzel sawenzel force-pushed the swenzel/handle_o2hitmerger_exitstatus branch 2 times, most recently from 23fa86b to 46a932c Compare June 12, 2025 12:10
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.
@sawenzel sawenzel force-pushed the swenzel/handle_o2hitmerger_exitstatus branch from 46a932c to b8f492f Compare June 12, 2025 12:11
@sawenzel sawenzel changed the title WIP: Improve exit handling of O2HitMerger Improve exit handling of O2HitMerger Jun 12, 2025
@alibuild
Copy link
Collaborator

alibuild commented Jun 12, 2025

Error while checking build/O2/fullCI_slc9 for b8f492f at 2025-06-12 20:16:

No log files found

Full log here.

@sawenzel sawenzel merged commit c4bca14 into AliceO2Group:dev Jun 13, 2025
10 of 11 checks passed
@sawenzel sawenzel deleted the swenzel/handle_o2hitmerger_exitstatus branch June 13, 2025 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants