Skip to content

Skip test_jemalloc_percpu_arena when CPU online override is ineffective#96300

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-jemalloc-percpu-arena-flaky
Feb 8, 2026
Merged

Skip test_jemalloc_percpu_arena when CPU online override is ineffective#96300
alexey-milovidov merged 1 commit intomasterfrom
fix-jemalloc-percpu-arena-flaky

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

The test bind-mounts a fake /sys/devices/system/cpu/online to simulate having fewer online CPUs than configured. However, sysconf(_SC_NPROCESSORS_ONLN) does not always read from that file - glibc can use sched_getaffinity instead, especially on newer kernels or ARM. When the override doesn't take effect, the test fails with assert online_cpus == 1 getting the real CPU count.

Skip instead of failing, since the test's precondition cannot be guaranteed on all systems.

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

The test bind-mounts a fake /sys/devices/system/cpu/online to simulate
having fewer online CPUs than configured. However,
`sysconf(_SC_NPROCESSORS_ONLN)` does not always read from that file -
glibc can use `sched_getaffinity` instead, especially on newer kernels
or ARM. When the override doesn't take effect, the test fails with
`assert online_cpus == 1` getting the real CPU count.

Skip instead of failing, since the test's precondition cannot be
guaranteed on all systems.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Feb 7, 2026

Workflow [PR], commit [1128a20]

Summary:

job_name test_name status info comment
Integration tests (amd_tsan, 6/6) failure
test_storage_s3_queue/test_1.py::test_max_set_size FAIL cidb
test_storage_s3_queue/test_1.py::test_s3_client_reused FAIL cidb
Integration tests (amd_asan, db disk, old analyzer, 3/6) error
Integration tests (amd_tsan, 3/6) error

@clickhouse-gh clickhouse-gh Bot added the pr-ci label Feb 7, 2026
@alexey-milovidov alexey-milovidov self-assigned this Feb 8, 2026
@alexey-milovidov alexey-milovidov merged commit 7a2bd1f into master Feb 8, 2026
129 of 134 checks passed
@alexey-milovidov alexey-milovidov deleted the fix-jemalloc-percpu-arena-flaky branch February 8, 2026 01:53
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 8, 2026
@shankar-iyer shankar-iyer added the post-approved Approved, but after the PR is merged. label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

post-approved Approved, but after the PR is merged. pr-ci pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants