Skip to content

Update tpls/cpr submodule to fix a hang on ALL_SANITIZERS_AVAILABLE when building a container#3021

Merged
annagrin merged 1 commit intoNVIDIA:mainfrom
annagrin:fix-out-of-memory
Jun 10, 2025
Merged

Update tpls/cpr submodule to fix a hang on ALL_SANITIZERS_AVAILABLE when building a container#3021
annagrin merged 1 commit intoNVIDIA:mainfrom
annagrin:fix-out-of-memory

Conversation

@annagrin
Copy link
Collaborator

@annagrin annagrin commented Jun 9, 2025

Fixes issue #2411

Description

Recently we have seen a hang (and eventually an OOM) when building a container, for example, using the following command:

➜  cuda-quantum git:(run-integration-test-in-ci)  docker build --build-arg install="CMAKE_BUILD_TYPE=Release CUDA_QUANTUM_VERSION=cu12-1"   --file docker/build/cudaq.dev.Dockerfile .
2025/06/09 12:52:39 in: []string{}
2025/06/09 12:52:39 Parsed entitlements: []
[+] Building 315.2s (8/9)                                                                                docker:default
 => [internal] load build definition from cudaq.dev.Dockerfile                                                     0.0s
 => => transferring dockerfile: 2.79kB                                                                             0.0s
 => [internal] load metadata for ghcr.io/nvidia/cuda-quantum-devdeps:ext-cu12.0-gcc11-main                         0.2s
 => [internal] load .dockerignore                                                                                  0.0s
 => => transferring context: 2B                                                                                    0.0s
 => [internal] load build context                                                                                  0.3s
 => => transferring context: 1.79MB                                                                                0.3s
 => [1/5] FROM ghcr.io/nvidia/cuda-quantum-devdeps:ext-cu12.0-gcc11-main@sha256:feb8a175d3796fe9651f9250e312a218f  0.0s
 => CACHED [2/5] ADD . /workspaces/cuda-quantum                                                                    0.0s
 => CACHED [3/5] WORKDIR /workspaces/cuda-quantum                                                                  0.0s
 => CACHED [4/5] RUN if [ -n "" ];     then         if [ ! -z "/usr/local/openmpi" ]; then             echo "Usin  0.0s
 => [5/5] RUN if [ -n "CMAKE_BUILD_TYPE=Release CUDA_QUANTUM_VERSION=cu12-1" ];     then         expected_prefi  314.6s
 => => # -- Performing Test ADDRESS_SANITIZER_AVAILABLE - Success
 => => # -- Performing Test LEAK_SANITIZER_AVAILABLE
 => => # -- Performing Test LEAK_SANITIZER_AVAILABLE - Success
 => => # -- Performing Test UNDEFINED_BEHAVIOUR_SANITIZER_AVAILABLE
 => => # -- Performing Test UNDEFINED_BEHAVIOUR_SANITIZER_AVAILABLE - Success
 => => # -- Performing Test ALL_SANITIZERS_AVAILABLE

The process tree shows a part of hyperv growing in memory:

image

Which seems to be caused by the following issue:

libcpr/cpr#1204

To fix the hang, this PR updates the tpls/cpr submodule to the commit that fixes the issue.

Signed-off-by: Anna Gringauze <agringauze@nvidia.com>
@annagrin annagrin requested a review from bmhowe23 June 9, 2025 22:08
@annagrin annagrin changed the title Update tpls/cpr submodule to fix a hang on TEST_ALL_SANITISERS when building a container Update tpls/cpr submodule to fix a hang on ALL_SANITIZERS_AVAILABLE when building a container Jun 9, 2025
@annagrin annagrin requested a review from 1tnguyen June 9, 2025 23:30
@annagrin annagrin enabled auto-merge (squash) June 10, 2025 00:00
Copy link
Collaborator

@1tnguyen 1tnguyen left a comment

Choose a reason for hiding this comment

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

Thanks @annagrin

@annagrin annagrin merged commit b89969d into NVIDIA:main Jun 10, 2025
195 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 10, 2025
@bettinaheim bettinaheim added the maintenance Work items to update and improve the code base label Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Work items to update and improve the code base

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants