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

[CI] Failure in ILMDocumentationIT.testAddSnapshotLifecyclePolicy #46021

Closed
original-brownbear opened this issue Aug 27, 2019 · 13 comments · Fixed by #46881
Closed

[CI] Failure in ILMDocumentationIT.testAddSnapshotLifecyclePolicy #46021

original-brownbear opened this issue Aug 27, 2019 · 13 comments · Fixed by #46881
Assignees
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >test-failure Triaged test failures from CI

Comments

@original-brownbear
Copy link
Member

The following failed on 7.x:

11:09:05 org.elasticsearch.client.documentation.ILMDocumentationIT > testAddSnapshotLifecyclePolicy FAILED
11:09:05     java.lang.AssertionError: 
11:09:05     Expected: <1L>
11:09:05          but: was <2L>
11:09:05         at __randomizedtesting.SeedInfo.seed([667E1B76EEB11627:FF88531C062144A]:0)
11:09:05         at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
11:09:05         at org.junit.Assert.assertThat(Assert.java:956)
11:09:05         at org.junit.Assert.assertThat(Assert.java:923)
11:09:05         at org.elasticsearch.client.documentation.ILMDocumentationIT.testAddSnapshotLifecyclePolicy(ILMDocumentationIT.java:869)
11:09:06 REPRODUCE WITH: ./gradlew :client:rest-high-level:integTestRunner --tests "org.elasticsearch.client.documentation.ILMDocumentationIT.testAddSnapshotLifecyclePolicy" -Dtests.seed=667E1B76EEB11627 -Dtests.security.manager=true -Dtests.locale=fr-LU -Dtests.timezone=America/Lima -Dcompiler.java=12 -Druntime.java=8

Build Scan

@dakrone can you take a look?

@original-brownbear original-brownbear added :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test-failure Triaged test failures from CI labels Aug 27, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@original-brownbear
Copy link
Member Author

@henningandersen
Copy link
Contributor

henningandersen commented Sep 2, 2019

Two more, also 7.x:

https://gradle-enterprise.elastic.co/s/ujxnu53ps7ms2/console-log?task=:client:rest-high-level:integTestRunner#L34
and
https://gradle-enterprise.elastic.co/s/kiaupjm4nqncu/console-log?task=:client:rest-high-level:integTestRunner

I wonder if it could be a variant of #46224 in that both cases seems to be either a delete/create or double create issue in rest tests.

@henningandersen
Copy link
Contributor

The symptom here looks sufficiently close to #46091, any objections to closing this?

@dakrone
Copy link
Member

dakrone commented Sep 10, 2019

@henningandersen it does look the same, let's close this for now and we can re-open if it occurs again now that we use ram disks for the CI.

@original-brownbear
Copy link
Member Author

This happened again here https://gradle-enterprise.elastic.co/s/ssgsd3wxbxvno in 7.x

@ywelsch ywelsch added :Data Management/ILM+SLM Index and Snapshot lifecycle management and removed :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs labels Sep 18, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@dnhatn
Copy link
Member

dnhatn commented Oct 23, 2019

This still happens after #46881: https://gradle-enterprise.elastic.co/s/5opdtqmvz3j4i.

@DaveCTurner
Copy link
Contributor

@gwbrown
Copy link
Contributor

gwbrown commented Oct 25, 2019

This is now the same problem as #48441: Could not determine repository generation from root blobs. We likely just need to add some retries around this type of exception, or wait until the underlying issue is fixed as described at #48433 (comment).

@original-brownbear
Copy link
Member Author

@gwbrown I think you'll be happier with retries and keeping the coverage for the time being. A fix (including reviews) will probably not happen in less than 2 weeks.

@gwbrown
Copy link
Contributor

gwbrown commented Oct 25, 2019

Thanks @original-brownbear, I was going to ping you soon to ask about the timeline on that. I'll move ahead with implementing retries as needed in these tests.

@gwbrown
Copy link
Contributor

gwbrown commented Oct 28, 2019

Now that #48548 has been merged and back ported, this failure should be fixed. I'm going to close this, please reopen if the failure shows up again.

@gwbrown gwbrown closed this as completed Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >test-failure Triaged test failures from CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants