Skip to content

02723_zookeeper_name: Force a deterministic result order#49594

Merged
alexey-milovidov merged 2 commits intomasterfrom
rs/02723_deterministic
May 6, 2023
Merged

02723_zookeeper_name: Force a deterministic result order#49594
alexey-milovidov merged 2 commits intomasterfrom
rs/02723_deterministic

Conversation

@rschu1ze
Copy link
Copy Markdown
Member

@rschu1ze rschu1ze commented May 6, 2023

Fixes flakiness seen e.g. here: https://s3.amazonaws.com/clickhouse-test-reports/49573/9cd81400f325b379fbb304c7e0da51c7e86e1143/stateless_tests__ubsan__[2_2].html

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 6, 2023
@robot-ch-test-poll3
Copy link
Copy Markdown
Contributor

robot-ch-test-poll3 commented May 6, 2023

This is an automated comment for commit d353cdc with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟡 pending

Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR🟡 pending
Mergeable CheckChecks if all other necessary checks are successful🟢 success
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub🟢 success

@alexey-milovidov alexey-milovidov self-assigned this May 6, 2023
@alexey-milovidov alexey-milovidov merged commit f0d0711 into master May 6, 2023
@alexey-milovidov alexey-milovidov deleted the rs/02723_deterministic branch May 6, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants