Skip to content

Disable CPU overload checks for all tests#80166

Merged
davenger merged 3 commits intomasterfrom
disable_cpu_overload_check_in_tests
May 14, 2025
Merged

Disable CPU overload checks for all tests#80166
davenger merged 3 commits intomasterfrom
disable_cpu_overload_check_in_tests

Conversation

@davenger
Copy link
Copy Markdown
Member

CPU overload detection contributes to flakiness of tests especially on slower debug and sanitizer builds.
Let's not enable this feature by default for all tests

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented May 13, 2025

Workflow [PR], commit [0976338]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 13, 2025
@nikitamikhaylov nikitamikhaylov self-assigned this May 13, 2025
@davenger davenger added this pull request to the merge queue May 14, 2025
Merged via the queue into master with commit f60cdb2 May 14, 2025
120 checks passed
@davenger davenger deleted the disable_cpu_overload_check_in_tests branch May 14, 2025 07:29
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-synced-to-cloud The PR is synced to the cloud repo label May 14, 2025
@Algunenano Algunenano self-assigned this May 14, 2025
robot-clickhouse-ci-2 added a commit that referenced this pull request May 14, 2025
Cherry pick #80166 to 25.4: Disable CPU overload checks for all tests
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-backports-created-cloud deprecated label, NOOP label May 14, 2025
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label May 14, 2025
Algunenano added a commit that referenced this pull request May 14, 2025
Backport #80166 to 25.4: Disable CPU overload checks for all tests
@robot-clickhouse robot-clickhouse added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-not-for-changelog This PR should not be mentioned in the changelog 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.

6 participants