Skip to content

[Common,PWGLF] Add forward compatibility with the new propagation-service-v2 - #17725

Open
ktf wants to merge 1 commit into
AliceO2Group:masterfrom
ktf:pr17725
Open

[Common,PWGLF] Add forward compatibility with the new propagation-service-v2#17725
ktf wants to merge 1 commit into
AliceO2Group:masterfrom
ktf:pr17725

Conversation

@ktf

@ktf ktf commented Sep 2, 2026

Copy link
Copy Markdown
Member

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 61 errors, ⚠️ 58 warnings, 🔕 0 disabled

@github-actions github-actions Bot changed the title Add forward compatibility with the new propagation-service-v2 [Common,PWGLF] Add forward compatibility with the new propagation-service-v2 Sep 2, 2026
@ktf ktf changed the title [Common,PWGLF] Add forward compatibility with the new propagation-service-v2 Add forward compatibility with the new propagation-service-v2 Sep 2, 2026
@github-actions github-actions Bot changed the title Add forward compatibility with the new propagation-service-v2 [Common,PWGLF] Add forward compatibility with the new propagation-service-v2 Sep 2, 2026
@ktf ktf changed the title [Common,PWGLF] Add forward compatibility with the new propagation-service-v2 Add forward compatibility with the new propagation-service-v2 Sep 2, 2026
@github-actions github-actions Bot changed the title Add forward compatibility with the new propagation-service-v2 [Common,PWGLF] Add forward compatibility with the new propagation-service-v2 Sep 2, 2026
@alibuild

alibuild commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 5f2d56a at 2026-09-03 15:32:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 23 errors and 68 warnings.'
Found 23 errors and 68 warnings.
++ [[ 23 -gt 0 ]]
++ cat /sw/BUILD/22e47d54d3ce6894ac6c314cf8e0cf53a830415d/O2Physics-code-check/errors.txt
Common/Core/TPCVDriftManager.h:111:11: error: variable 'tTB' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
Common/Core/TPCVDriftManager.h:111:16: error: variable 'tTBErr' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
Common/Core/TPCVDriftManager.h:115:7: error: uninitialized record type: 'enc' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
Common/Core/TPCVDriftManager.h:116:20: error: do not access members of unions; consider using (boost::)variant instead [cppcoreguidelines-pro-type-union-access,-warnings-as-errors]
Common/Tools/TrackTuner.h:392:19: error: the variable 'str' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference [performance-unnecessary-copy-initialization,-warnings-as-errors]
Common/Tools/TrackTuner.h:599:17: error: unused local variable 'grOneOverPtPionNameMC' of type 'std::string' (aka 'basic_string<char>') [bugprone-unused-local-non-trivial-variable,-warnings-as-errors]
Common/Tools/TrackTuner.h:600:17: error: unused local variable 'grOneOverPtPionNameData' of type 'std::string' (aka 'basic_string<char>') [bugprone-unused-local-non-trivial-variable,-warnings-as-errors]
Common/Tools/TrackTuner.h:629:50: error: Called C++ object pointer is null [clang-analyzer-core.CallAndMessage,-warnings-as-errors]
Common/Tools/TrackTuner.h:762:18: error: narrowing conversion from 'double' to 'int' [bugprone-narrowing-conversions,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:60:9: error: function-like macro 'BITSET' used; consider a 'constexpr' template function [cppcoreguidelines-macro-usage,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:322:80: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:408:85: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:419:95: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:423:91: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:424:99: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:425:91: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:426:91: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:453:3: error: constructor does not initialize these fields: mRunNumber [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:558:10: error: constructor does not initialize these fields: xyz, posP, negP, momentum [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:577:10: error: constructor does not initialize these fields: label, motherLabel, mcCollision, pdgCode, pdgCodeMother, pdgCodeV0, pdgCodePositive, pdgCodeNegative, pdgCodeBachelor, isPhysicalPrimary, xyz, lxyz, posP, negP, bachP, momentum, mcParticlePositive, mcParticleNegative, mcParticleBachelor [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:1708:11: error: uninitialized record type: 'positivePositionIU' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:1709:11: error: uninitialized record type: 'negativePositionIU' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/Utils/strangenessBuilderModule.h:2768:7: error: uninitialized record type: 'dcaInfo' [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
++ [[ 68 -gt 0 ]]
++ cat /sw/BUILD/22e47d54d3ce6894ac6c314cf8e0cf53a830415d/O2Physics-code-check/warnings.txt
Common/Tools/TrackPropagationModule.h:59:1: warning: nested namespaces can be concatenated [modernize-concat-nested-namespaces]
Common/Tools/TrackPropagationModule.h:89:3: warning: use '= default' to define a trivial default constructor [modernize-use-equals-default]
Common/Tools/TrackPropagationModule.h:164:17: warning: redundant string initialization [readability-redundant-string-init]
Common/Tools/TrackPropagationModule.h:319:7: warning: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto]
Common/Tools/TrackTuner.h:88:15: warning: redundant string initialization [readability-redundant-string-init]
Common/Tools/TrackTuner.h:89:15: warning: redundant string initialization [readability-redundant-string-init]
Common/Tools/TrackTuner.h:90:15: warning: redundant string initialization [readability-redundant-string-init]
Common/Tools/TrackTuner.h:91:15: warning: redundant string initialization [readability-redundant-string-init]
Common/Tools/TrackTuner.h:102:15: warning: redundant string initialization [readability-redundant-string-init]
Common/Tools/TrackTuner.h:373:40: warning: statement should be inside braces [readability-braces-around-statements]
Common/Tools/TrackTuner.h:402:12: warning: do not use 'compare' to test equality of strings; use the string equality operator instead [readability-string-compare]
Common/Tools/TrackTuner.h:404:19: warning: do not use 'compare' to test equality of strings; use the string equality operator instead [readability-string-compare]
Common/Tools/TrackTuner.h:415:60: warning: implicit conversion 'bool' -> 'int' [readability-implicit-bool-conversion]
Common/Tools/TrackTuner.h:420:53: warning: implicit conversion 'bool' -> 'int' [readability-implicit-bool-conversion]
Common/Tools/TrackTuner.h:424:59: warning: implicit conversion 'bool' -> 'int' [readability-implicit-bool-conversion]
Common/Tools/TrackTuner.h:428:61: warning: implicit conversion 'bool' -> 'int' [readability-implicit-bool-conversion]
Common/Tools/TrackTuner.h:432:59: warning: implicit conversion 'bool' -> 'int' [readability-implicit-bool-conversion]
[0 more errors; see full log]

Full log here.

@alibuild

alibuild commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 5f2d56a at 2026-09-04 10:29:

## sw/BUILD/O2Physics-latest/log
2026-09-04T00:41:36.346+0200 [24742:140087398428288] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.GetActionResult()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T00:41:36.352+0200 [24742:140087398428288] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8980": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame
2026-09-04T00:44:47.644+0200 [24742:140087398428288] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "FindMissingBlobs()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T00:45:12.629+0200 [24780:140019455614592] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.GetActionResult()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T00:45:12.629+0200 [24780:140019455614592] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8980": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame
2026-09-04T05:37:40.334+0200 [43943:140343578582656] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T05:37:40.352+0200 [43943:140343578582656] [executioncontext.cpp:739] [ERROR] Failed to download output blob 80016b7dc387e8b90c904c7f529a193b1e7bb413643d24b02b1ce9b0a4bf6540: 14: Socket closed
2026-09-04T05:50:24.929+0200 [44960:140554466729600] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.GetActionResult()", last gRPC error was [14: Socket closed]
2026-09-04T05:50:24.930+0200 [44960:140554466729600] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8980": 14: Socket closed
2026-09-04T05:52:34.419+0200 [44960:140554466729600] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "FindMissingBlobs()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T05:51:02.590+0200 [44933:140386996375168] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.GetActionResult()", last gRPC error was [14: Socket closed]
2026-09-04T05:51:02.591+0200 [44933:140386996375168] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8980": 14: Socket closed
2026-09-04T05:53:09.309+0200 [44933:140386996375168] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "FindMissingBlobs()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T06:04:44.280+0200 [44102:140172199718528] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T06:04:44.285+0200 [44102:140172199718528] [executioncontext.cpp:739] [ERROR] Failed to download output blob e0dc094bb29cd07a54b29a50c126b6c5113c1e48cb31e859be888d7f00fa847b: 14: Socket closed
2026-09-04T06:05:01.368+0200 [44076:140400737693312] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T06:05:01.381+0200 [44076:140400737693312] [executioncontext.cpp:739] [ERROR] Failed to download output blob dd2448597d35ebed28ef5ecde1c9ab06e9ab4905da5f6c80bf39460b8e349827: 14: Socket closed
2026-09-04T06:29:56.236+0200 [44187:139729151385216] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "BatchUpdateBlobs()", last gRPC error was [14: Socket closed]
2026-09-04T06:30:16.240+0200 [44187:139729151385216] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.UpdateActionResult()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T06:32:22.701+0200 [44682:140109466848896] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "FindMissingBlobs()", last gRPC error was [14: Socket closed]
2026-09-04T07:22:39.126+0200 [43020:139975809900160] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T07:22:39.153+0200 [43020:139975809900160] [executioncontext.cpp:739] [ERROR] Failed to download output blob 6ea4c310a1565d0b5d699e23336d699e46af6e5f71a59b4db17166aa2b360c9c: 14: Socket closed
2026-09-04T09:05:59.888+0200 [44982:139654199967360] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ActionCache.GetActionResult()", last gRPC error was [14: Socket closed]
2026-09-04T09:05:59.888+0200 [44982:139654199967360] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8980": 14: Socket closed
2026-09-04T09:08:33.796+0200 [44982:139654199967360] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "FindMissingBlobs()", last gRPC error was [14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8980: connection attempt timed out before receiving SETTINGS frame]
2026-09-04T09:27:13.088+0200 [44641:140630499511936] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T09:27:13.089+0200 [44641:140630499511936] [executioncontext.cpp:739] [ERROR] Failed to download output blob 60e4342ac119db6ef590d7997eb16465aeb9cabeda0efa9bf6a6688de9db2cc4: 14: Socket closed
2026-09-04T10:21:33.085+0200 [44255:139673788952192] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T10:21:33.092+0200 [44255:139673788952192] [executioncontext.cpp:739] [ERROR] Failed to download output blob 3f56dacb5d3cf9d2b2e590b9f308145059bcdd23e90f84d3afffc0632aad0dac: 14: Socket closed
2026-09-04T10:25:06.934+0200 [44030:139914591641216] [buildboxcommon_grpcretrier.cpp:177] [ERROR] Retry limit (0) exceeded for "ByteStream.Read()", last gRPC error was [14: Socket closed]
2026-09-04T10:25:06.944+0200 [44030:139914591641216] [executioncontext.cpp:739] [ERROR] Failed to download output blob 35d49b1301522701d12b33cd039b562eb7b36ebe63cc629f2618bfadf3de35c9: 14: Socket closed

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants