Skip to content

[CI] PeerRecoveryTargetServiceTests testPrepareIndexForPeerRecovery failure #52221

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
Closed
davidkyle opened this issue Feb 11, 2020 · 1 comment · Fixed by #52245
Closed

[CI] PeerRecoveryTargetServiceTests testPrepareIndexForPeerRecovery failure #52221

davidkyle opened this issue Feb 11, 2020 · 1 comment · Fixed by #52245
Assignees
@dnhatn
Labels
:Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. >test-failure Triaged test failures from CI v8.0.0-alpha1

Comments

@davidkyle
Copy link
Member 

java.lang.AssertionError: 0 > -1
	at __randomizedtesting.SeedInfo.seed([9ADBDC3EA699F935:41553D6437F6AF95]:0)
	at org.elasticsearch.index.translog.Translog.newSnapshot(Translog.java:615)
	at org.elasticsearch.indices.recovery.PeerRecoveryTargetServiceTests.testPrepareIndexForPeerRecovery(PeerRecoveryTargetServiceTests.java:180)

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+release-tests/888/console
https://gradle-enterprise.elastic.co/s/hjg6zbffzaoxu

./gradlew ':server:test' --tests "org.elasticsearch.indices.recovery.PeerRecoveryTargetServiceTests.testPrepareIndexForPeerRecovery" \
  -Dtests.seed=9ADBDC3EA699F935 \
  -Dtests.security.manager=true \
  -Dbuild.snapshot=false \
  -Dtests.jvm.argline="-Dbuild.snapshot=false" \
  -Dtests.locale=es-HN \
  -Dtests.timezone=Africa/Dakar \
  -Dcompiler.java=13
@davidkyle davidkyle added :Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. v8.0.0 labels Feb 11, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Recovery)

@davidkyle davidkyle added the >test-failure Triaged test failures from CI label Feb 11, 2020
@dnhatn dnhatn mentioned this issue Feb 11, 2020
Merged
@dnhatn dnhatn self-assigned this Feb 11, 2020
dnhatn added a commit that referenced this issue Feb 12, 2020
@dnhatn
Fix testPrepareIndexForPeerRecovery (#52245)
0c7ae02 
Since #51905, we skip translog recovery if the local checkpoint of the 
safe commit equals to the global checkpoint. This change adjusts the
test not to create a new snapshot in that case.

Closes #52221
Relates #51905
dnhatn added a commit to dnhatn/elasticsearch that referenced this issue Feb 26, 2020
@dnhatn
Fix testPrepareIndexForPeerRecovery (elastic#52245)
c0c0200 
Since elastic#51905, we skip translog recovery if the local checkpoint of the
safe commit equals to the global checkpoint. This change adjusts the
test not to create a new snapshot in that case.

Closes elastic#52221
Relates elastic#51905
dnhatn added a commit that referenced this issue Feb 26, 2020
@dnhatn
Fix testPrepareIndexForPeerRecovery (#52245)
d7fe135 
Since #51905, we skip translog recovery if the local checkpoint of the
safe commit equals to the global checkpoint. This change adjusts the
test not to create a new snapshot in that case.

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

@dnhatn dnhatn

Labels
:Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. >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.

Fix testPrepareIndexForPeerRecovery dnhatn/elasticsearch
4 participants
@jakelandis @davidkyle @dnhatn @elasticmachine