Skip to content

Conversation

@dakrone
Copy link
Member

@dakrone dakrone commented Jan 31, 2020

This previously was missing some key information in the output of the failure. This captures that
information and adds logging at each step so we can determine the cause if it fails again.

Resolves #50358

This previously was missing some key information in the output of the failure. This captures that
information and adds logging at each step so we can determine the cause *if* it fails again.

Resolves elastic#50358
@dakrone dakrone added >test Issues or PRs that are addressing/adding tests :Data Management/ILM+SLM Index and Snapshot lifecycle management v8.0.0 v7.7.0 labels Jan 31, 2020
@dakrone dakrone requested a review from andreidan January 31, 2020 18:48
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@andreidan andreidan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @dakrone

@dakrone dakrone merged commit ae8017a into elastic:master Jan 31, 2020
@dakrone dakrone deleted the ilm-fix-full-policy-again branch January 31, 2020 20:01
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Jan 31, 2020
* Fix SnapshotLifecycleRestIT.testFullPolicySnapshot

This previously was missing some key information in the output of the failure. This captures that
information and adds logging at each step so we can determine the cause *if* it fails again.

Resolves elastic#50358
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 Issues or PRs that are addressing/adding tests v7.7.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SnapshotLifecycleRestIT testFullPolicySnapshot failure

4 participants