DO NOT MERGE fix(ci): add shmem size for cmapi test container#3983
DO NOT MERGE fix(ci): add shmem size for cmapi test container#3983mariadb-LeonidFedorov wants to merge 3 commits into
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request increases the shared memory size for cmapi test containers to 500MB to prevent exhaustion during ASAN builds and adds documentation explaining the change. A review comment suggests extending this configuration to upgrade and regression containers to ensure consistency and reliability across all ColumnStore-related test stages.
| # --shm-size: ColumnStore BRM (ExtentMap, VSS, VBBM, CopyLocks) uses | ||
| # /dev/shm for SysV/POSIX shared memory. Docker default is 64MiB which | ||
| # is easily exhausted under ASAN (2-3x memory footprint), especially | ||
| # when cmapi step runs in parallel with upgrade/mtr/regression steps | ||
| # on the same pipeline host. | ||
| docker_run_args+=(--shm-size=500m --env PYTHONPATH="${PYTHONPATH}") |
There was a problem hiding this comment.
The comment on lines 79-83 explicitly mentions that cmapi runs in parallel with upgrade and regression steps, and that the shared memory exhaustion is a known issue for ColumnStore components (BRM) under ASAN. While mtr and cmapi now have --shm-size=500m configured, the upgrade (line 85) and regression (line 87) container blocks appear to still use the default 64MiB limit. If these test stages also utilize ColumnStore, they are likely to suffer from the same intermittent failures. Consider applying the increased shared memory size to those container types as well for consistency and CI reliability.
mariadb-LeonidFedorov
changed the title
mariadb-LeonidFedorov
force-pushed
the
cmapi-shmem
branch
from
02982f9 to
7bca70e
Compare
1 participant
No description provided.