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

[ILM] TimeSeriesLifecycleActionsIT.testRolloverAlreadyExists failure #49073

Closed
dimitris-athanasiou opened this issue Nov 14, 2019 · 1 comment · Fixed by #49104
Closed

[ILM] TimeSeriesLifecycleActionsIT.testRolloverAlreadyExists failure #49073

dimitris-athanasiou opened this issue Nov 14, 2019 · 1 comment · Fixed by #49104
Assignees
@andreidan
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >test-failure Triaged test failures from CI

Comments

@dimitris-athanasiou
Copy link
Contributor

Console log: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+multijob-unix-compatibility/os=oraclelinux-7&&immutable/390/console

Build scan: https://gradle-enterprise.elastic.co/s/2knxuomyu6d62

Failure:

ava.lang.AssertionError: 

Expected: <{"phase":"hot","action":"rollover","name":"ERROR"}>
     but: was <{"phase":"hot","action":"rollover","name":"check-rollover-ready"}>
Close stacktrace
at __randomizedtesting.SeedInfo.seed([2359606ACADA06B8:3C3BEB9E601FCD94]:0)
at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
at org.junit.Assert.assertThat(Assert.java:956)
at org.junit.Assert.assertThat(Assert.java:923)
at org.elasticsearch.xpack.ilm.TimeSeriesLifecycleActionsIT.lambda$testRolloverAlreadyExists$17(TimeSeriesLifecycleActionsIT.java:318)
at org.elasticsearch.test.ESTestCase.assertBusy(ESTestCase.java:874)
at org.elasticsearch.test.ESTestCase.assertBusy(ESTestCase.java:847)
at org.elasticsearch.xpack.ilm.TimeSeriesLifecycleActionsIT.testRolloverAlreadyExists(TimeSeriesLifecycleActionsIT.java:314)

Reproduce with:

./gradlew ':x-pack:plugin:ilm:qa:multi-node:integTestRunner' --tests "org.elasticsearch.xpack.ilm.TimeSeriesLifecycleActionsIT.testRolloverAlreadyExists" -Dtests.seed=2359606ACADA06B8 -Dtests.security.manager=true -Dtests.locale=sr -Dtests.timezone=America/Atikokan -Dcompiler.java=12 -Druntime.java=11

I could not reproduce locally.
@dimitris-athanasiou dimitris-athanasiou added >test-failure Triaged test failures from CI :Data Management/ILM+SLM Index and Snapshot lifecycle management labels Nov 14, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/ILM+SLM)

dimitris-athanasiou added a commit to dimitris-athanasiou/elasticsearch that referenced this issue Nov 14, 2019
@dimitris-athanasiou
[ILM] Mute TimeSeriesLifecycleActionsIT.testRolloverAlreadyExists
81d01b6 
Relates elastic#49073
@dimitris-athanasiou dimitris-athanasiou mentioned this issue Nov 14, 2019
Merged
dimitris-athanasiou added a commit that referenced this issue Nov 14, 2019
@dimitris-athanasiou
[ILM] Mute TimeSeriesLifecycleActionsIT.testRolloverAlreadyExists (#4…
736651a 
…9074)

Relates #49073
@andreidan andreidan self-assigned this Nov 14, 2019
andreidan added a commit to andreidan/elasticsearch that referenced this issue Nov 14, 2019
@andreidan
ILM Remove obsolete testRolloverAlreadyExists
2bdbc54 
The rollover action is now a retryable step (see elastic#48256)
so ILM will keep retrying until it succeeds as opposed to stopping and
moving the execution in the ERROR step.

Fixes elastic#49073
@andreidan andreidan mentioned this issue Nov 14, 2019
Merged
andreidan added a commit that referenced this issue Nov 15, 2019
@andreidan
ILM Remove obsolete testRolloverAlreadyExists (#49104)
3ae9089 
The rollover action is now a retryable step (see #48256)
so ILM will keep retrying until it succeeds as opposed to stopping and
moving the execution in the ERROR step.

Fixes #49073
andreidan added a commit to andreidan/elasticsearch that referenced this issue Nov 15, 2019
@andreidan
ILM Remove obsolete testRolloverAlreadyExists (elastic#49104)
590fa2e 
The rollover action is now a retryable step (see elastic#48256)
so ILM will keep retrying until it succeeds as opposed to stopping and
moving the execution in the ERROR step.

Fixes elastic#49073

(cherry picked from commit 3ae9089)
Signed-off-by: Andrei Dan <andrei.dan@elastic.co>

# Conflicts:
#	x-pack/plugin/ilm/qa/multi-node/src/test/java/org/elasticsearch/xpack/ilm/TimeSeriesLifecycleActionsIT.java
@andreidan andreidan mentioned this issue Nov 15, 2019
Merged
andreidan added a commit that referenced this issue Nov 15, 2019
@andreidan
ILM Remove obsolete testRolloverAlreadyExists (#49104) (#49144)
085d08c 
The rollover action is now a retryable step (see #48256)
so ILM will keep retrying until it succeeds as opposed to stopping and
moving the execution in the ERROR step.

Fixes #49073

(cherry picked from commit 3ae9089)
Signed-off-by: Andrei Dan <andrei.dan@elastic.co>

# Conflicts:
#	x-pack/plugin/ilm/qa/multi-node/src/test/java/org/elasticsearch/xpack/ilm/TimeSeriesLifecycleActionsIT.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andreidan andreidan

Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ILM Remove obsolete testRolloverAlreadyExists andreidan/elasticsearch
3 participants
@andreidan @dimitris-athanasiou @elasticmachine