Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mute FullClusterRestartTest#testWatcher and 30s timeout for ye… #48848

Merged

Conversation

jakelandis
Copy link
Contributor

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: #48381
Related: #48434
Related: #47950
Related: #40178

…r yellow

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: elastic#48381
Related: elastic#48434
Related: elastic#47950
Related: elastic#40178
@jakelandis jakelandis added >test Issues or PRs that are addressing/adding tests :Data Management/Watcher v8.0.0 v7.5.0 v7.6.0 labels Nov 4, 2019
@jakelandis jakelandis requested a review from danhermann November 4, 2019 16:41
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Watcher)

Copy link
Contributor

@danhermann danhermann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jakelandis jakelandis changed the title Mute FullClusterRestartTest#testWatcher and 30s timeout for yellow Mute FullClusterRestartTest#testWatcher and 30s timeout for ye… Nov 4, 2019
@jakelandis jakelandis merged commit 3aa44e4 into elastic:master Nov 4, 2019
@jakelandis jakelandis deleted the re-mute-watcher-and-rollback-to30s branch November 4, 2019 17:35
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Nov 4, 2019
…tic#48848)

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: elastic#48381
Related: elastic#48434
Related: elastic#47950
Related: elastic#40178
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Nov 4, 2019
…tic#48848)

The timeout was increased to 60s to allow this test more time to reach a
yellow state. However, the test will still on occasion fail even with the
60s timeout.

Related: elastic#48381
Related: elastic#48434
Related: elastic#47950
Related: elastic#40178
@ywelsch
Copy link
Contributor

ywelsch commented Feb 27, 2020

The backport PR seems to have been merged. I'm therefore removing the backport pending label here. Please shout if this is incorrect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Watcher >test Issues or PRs that are addressing/adding tests v7.5.0 v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants