Skip to content

[PWGLF] bug fix in phi analysis#16145

Merged
dmallick2 merged 5 commits into
AliceO2Group:masterfrom
sarjeetagami:master
May 10, 2026
Merged

[PWGLF] bug fix in phi analysis#16145
dmallick2 merged 5 commits into
AliceO2Group:masterfrom
sarjeetagami:master

Conversation

@sarjeetagami

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the pwglf label May 7, 2026
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

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

@sarjeetagami sarjeetagami marked this pull request as ready for review May 7, 2026 13:08
@alibuild

alibuild commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 322071d at 2026-05-07 15:09:

## sw/BUILD/O2Physics-latest/log
CMake Error at /sw/slc9_x86-64/CMake/v4.1.4-2/share/cmake-4.1/Modules/CMakeTestCCompiler.cmake:67 (message):
    2026-05-07T15:09:13.827+0200 [69:140431164280448] [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:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T15:09:13.828+0200 [69:140431164280448] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused
    2026-05-07T15:09:13.828+0200 [69:140431164280448] [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: ipv6:%5B::1%5D:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T15:09:13.828+0200 [69:140431164280448] [executioncontext.cpp:908] [ERROR] Error while uploading resources to CAS at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv6:%5B::1%5D:8085: Failed to connect to remote host: Connection refused
    ninja: build stopped: subcommand failed.

Full log here.

Comment thread PWGLF/Tasks/Resonances/phianalysisrun3_PbPb.cxx Outdated
@vkucera vkucera marked this pull request as draft May 7, 2026 13:51
@sarjeetagami sarjeetagami marked this pull request as ready for review May 7, 2026 14:23
@alibuild

alibuild commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 51d7517 at 2026-05-07 16:29:

## sw/BUILD/O2Physics-latest/log
CMake Error at /sw/slc9_x86-64/CMake/v4.1.4-2/share/cmake-4.1/Modules/CMakeTestCCompiler.cmake:67 (message):
    2026-05-07T16:29:38.503+0200 [70:140610560556672] [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:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T16:29:38.504+0200 [70:140610560556672] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused
    2026-05-07T16:29:38.504+0200 [70:140610560556672] [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:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T16:29:38.504+0200 [70:140610560556672] [executioncontext.cpp:908] [ERROR] Error while uploading resources to CAS at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused
    ninja: build stopped: subcommand failed.

Full log here.

@sarjeetagami sarjeetagami marked this pull request as draft May 7, 2026 14:32
@sarjeetagami sarjeetagami reopened this May 7, 2026
@sarjeetagami sarjeetagami marked this pull request as ready for review May 7, 2026 17:24
@alibuild

alibuild commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 9a75ea7 at 2026-05-07 21:41:

## sw/BUILD/O2Physics-latest/log
CMake Error at /sw/slc9_x86-64/CMake/v4.1.4-2/share/cmake-4.1/Modules/CMakeTestCCompiler.cmake:67 (message):
    2026-05-07T21:41:29.265+0200 [70:140273613213312] [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:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T21:41:29.265+0200 [70:140273613213312] [executioncontext.cpp:544] [ERROR] Error while querying action cache at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv4:127.0.0.1:8085: Failed to connect to remote host: Connection refused
    2026-05-07T21:41:29.266+0200 [70:140273613213312] [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: ipv6:%5B::1%5D:8085: Failed to connect to remote host: Connection refused]
    2026-05-07T21:41:29.266+0200 [70:140273613213312] [executioncontext.cpp:908] [ERROR] Error while uploading resources to CAS at "http://localhost:8085": 14: failed to connect to all addresses; last error: UNKNOWN: ipv6:%5B::1%5D:8085: Failed to connect to remote host: Connection refused
    ninja: build stopped: subcommand failed.

Full log here.

prottayCMT
prottayCMT previously approved these changes May 7, 2026
Comment thread PWGLF/Tasks/Resonances/phianalysisrun3_PbPb.cxx Outdated
@alibuild

alibuild commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 4cbc4d4 at 2026-05-08 08:59:

No log files found

Full log here.

@vkucera

vkucera commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Don't force push. It's written in the guidelines.

@vkucera

vkucera commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Please fix the remaining errors. Both the struct name and the device name you provide are invalid, while you are not supposed to provide the device name at all.

@sarjeetagami sarjeetagami marked this pull request as draft May 9, 2026 12:18
@sarjeetagami sarjeetagami marked this pull request as ready for review May 9, 2026 13:45
@sarjeetagami

Copy link
Copy Markdown
Contributor Author

@vkucera, All comments have been incorporated. The remaining linter error is not from my task; it is coming from the CMakeLists file. Could you please approve the changes now?

@vkucera

vkucera commented May 9, 2026

Copy link
Copy Markdown
Collaborator

@vkucera, All comments have been incorporated. The remaining linter error is not from my task; it is coming from the CMakeLists file. Could you please approve the changes now?

Thanks @sarjeetagami for the fixes. You just have 4 warnings left: A missing line after the copyright header and 3 constants that need the k prefix to be removed. Can you also fix those?

@sarjeetagami

Copy link
Copy Markdown
Contributor Author

@vkucera, All O2 linter errors and warnings have been fixed, thank you!

@vkucera vkucera left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@sarjeetagami Thanks for the fixes.

@dmallick2 dmallick2 merged commit 412d07b into AliceO2Group:master May 10, 2026
13 of 15 checks passed
SuJeong-Ji pushed a commit to SuJeong-Ji/O2Physics that referenced this pull request May 22, 2026
Co-authored-by: sarjeeta gami <sarjeetagami@sarjeetas-MacBook-Air-217.local>
jjuracka pushed a commit to jjuracka/O2Physics that referenced this pull request Jun 6, 2026
Co-authored-by: sarjeeta gami <sarjeetagami@sarjeetas-MacBook-Air-217.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

5 participants