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.

Sign up for GitHub

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

Jump to bottom

RecoveryDuringReplicationTests.testDoNotWaitForPendingSeqNo failed #39510

Closed
tlrx opened this issue Feb 28, 2019 · 3 comments · Fixed by #39595
Closed

RecoveryDuringReplicationTests.testDoNotWaitForPendingSeqNo failed #39510

tlrx opened this issue Feb 28, 2019 · 3 comments · Fixed by #39595
Assignees
@dnhatn
Labels
:Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >test-failure Triaged test failures from CI v8.0.0-alpha1

Comments

@tlrx
Copy link
Member

tlrx commented Feb 28, 2019

The test RecoveryDuringReplicationTests.testDoNotWaitForPendingSeqNo() failed on CI:
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+matrix-java-periodic/ES_BUILD_JAVA=java11,ES_RUNTIME_JAVA=java11,nodes=immutable&&linux&&docker/267/consoleFull

It does not reproduce with:

./gradlew :server:unitTest \
  -Dtests.seed=F8E8B6147433686D \
  -Dtests.class=org.elasticsearch.index.replication.RecoveryDuringReplicationTests \
  -Dtests.security.manager=true \
  -Dtests.locale=en-US \
  -Dtests.timezone=UTC \
  -Dcompiler.java=11 \
  -Druntime.java=11

Relevant log:
consoleText.txt
@tlrx tlrx added :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >test-failure Triaged test failures from CI v8.0.0 labels Feb 28, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@dnhatn dnhatn self-assigned this Feb 28, 2019
@gwbrown
Copy link
Contributor

gwbrown commented Mar 1, 2019

Failed again today https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+7.x+periodic/137/console

Looks like it fails ~1-3 times/day

@dnhatn dnhatn mentioned this issue Mar 1, 2019
Merged
@dnhatn
Copy link
Member

dnhatn commented Mar 1, 2019

I opened #39595.

DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this issue Mar 2, 2019
@DaveCTurner
Mute testDoNotWaitForPendingSeqNo
9f130a5 
Relates elastic#39510, elastic#39595.
dnhatn pushed a commit that referenced this issue Mar 4, 2019
@DaveCTurner @dnhatn
Mute testDoNotWaitForPendingSeqNo
9ec24ba 
Relates #39510, #39595.
dnhatn pushed a commit that referenced this issue Mar 4, 2019
@DaveCTurner @dnhatn
Mute testDoNotWaitForPendingSeqNo
3aa24b1 
Relates #39510, #39595.
dnhatn pushed a commit that referenced this issue Mar 4, 2019
@DaveCTurner @dnhatn
Mute testDoNotWaitForPendingSeqNo
6d2c886 
Relates #39510, #39595.
dnhatn added a commit that referenced this issue Mar 4, 2019
@dnhatn
Assert recovery done in testDoNotWaitForPendingSeqNo (#39595)
7fe62d0 
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510
dnhatn added a commit that referenced this issue Mar 4, 2019
@dnhatn
Assert recovery done in testDoNotWaitForPendingSeqNo (#39595)
801f13f 
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510
dnhatn added a commit that referenced this issue Mar 4, 2019
@dnhatn
Assert recovery done in testDoNotWaitForPendingSeqNo (#39595)
b24b73c 
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510
dnhatn added a commit that referenced this issue Mar 4, 2019
@dnhatn
Assert recovery done in testDoNotWaitForPendingSeqNo (#39595)
0799ff1 
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

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

@dnhatn dnhatn

Labels
:Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >test-failure Triaged test failures from CI v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Assert recovery done in testDoNotWaitForPendingSeqNo dnhatn/elasticsearch
5 participants
@tlrx @jakelandis @gwbrown @dnhatn @elasticmachine