Skip to content

Conversation

@jakelandis
Copy link
Contributor

The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.


As part of re-enabling via #47950 this test was run for 24+ hours without failure on a dedicated node. I believe that this timeout is an environmental issue and 60s should be plenty of time.

The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
@jakelandis jakelandis added >test Issues or PRs that are addressing/adding tests :Data Management/Watcher v8.0.0 v7.6.0 labels Oct 23, 2019
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@martijnvg martijnvg 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 merged commit 28bfa6e into elastic:master Oct 24, 2019
@jakelandis jakelandis deleted the increase_timeout_for_yellow branch October 24, 2019 13:50
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Oct 24, 2019
The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Oct 24, 2019
The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
jakelandis added a commit that referenced this pull request Oct 24, 2019
…8480)

The testWatcher method will on occasion timeout waiting for
a yellow cluster state. This change increases the timeout
to 60s.
jakelandis added a commit to jakelandis/elasticsearch that referenced this pull request Nov 4, 2019
…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 added a commit that referenced this pull request Nov 4, 2019
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
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
jakelandis added a commit that referenced this pull request Nov 11, 2019
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
jakelandis added a commit that referenced this pull request Nov 11, 2019
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
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.

3 participants