Skip to content

Disable flacky test #8840#11320

Merged
alexey-milovidov merged 1 commit intomasterfrom
revert-revert-disable-flacky-tests
May 31, 2020
Merged

Disable flacky test #8840#11320
alexey-milovidov merged 1 commit intomasterfrom
revert-revert-disable-flacky-tests

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov commented May 31, 2020

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

This commit partially reverts #8840
Note that MOVE TTL is considered experimental (not for production usage) before these tests will be enabled. @excitoon

Update: I have read that test and figured out that it's totally wrong (synchronization with sleeps) and should not exist at all.

@blinkov blinkov added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 31, 2020
@alexey-milovidov alexey-milovidov added the testing Special issue with list of bugs found by CI label May 31, 2020
@alexey-milovidov alexey-milovidov merged commit 92c56af into master May 31, 2020
@alexey-milovidov alexey-milovidov deleted the revert-revert-disable-flacky-tests branch May 31, 2020 21:10
@alexey-milovidov
Copy link
Copy Markdown
Member Author

33 checks passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-docs-needed pr-not-for-changelog This PR should not be mentioned in the changelog testing Special issue with list of bugs found by CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants