Skip to content

tests: fix test_quorum_inserts flakiness#77160

Merged
azat merged 1 commit intoClickHouse:masterfrom
azat:tests/test_quorum_inserts
Mar 5, 2025
Merged

tests: fix test_quorum_inserts flakiness#77160
azat merged 1 commit intoClickHouse:masterfrom
azat:tests/test_quorum_inserts

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Mar 5, 2025

Fix the unpredictible nature of
test_insert_quorum_with_keeper_loss_connection, that leads to failures of other tests due to some failpoints leaved as active.

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Fixes: #61953
Fixes: #67604

@azat azat requested a review from hanfei1991 March 5, 2025 12:59
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Mar 5, 2025

Workflow [PR], commit [3728eb5]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Mar 5, 2025
@hanfei1991 hanfei1991 self-assigned this Mar 5, 2025
@azat azat force-pushed the tests/test_quorum_inserts branch from b0f51e0 to 85b09e8 Compare March 5, 2025 15:34
Fix the unpredictible nature of
test_insert_quorum_with_keeper_loss_connection, that leads to failures
of other tests due to some failpoints leaved as active.
@azat azat force-pushed the tests/test_quorum_inserts branch from 85b09e8 to 3728eb5 Compare March 5, 2025 15:35
@azat azat added the 🍃 green ci 🌿 Fixing flaky tests in CI label Mar 5, 2025
@azat azat added this pull request to the merge queue Mar 5, 2025
Merged via the queue into ClickHouse:master with commit a730d71 Mar 5, 2025
124 checks passed
@azat azat deleted the tests/test_quorum_inserts branch March 5, 2025 20:20
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍃 green ci 🌿 Fixing flaky tests in CI 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.

Test test_quorum_inserts/test.py is flaky

3 participants