Skip to content

Conversation

@dakrone
Copy link
Member

@dakrone dakrone commented Jan 24, 2020

Backports the following commits to 7.x:

…c#51386)

* Check all snapshots in SnapshotLifecycleRestIT.testFullPolicy

Rather than check the first returned snapshot for a snapshot starting with `snap-` in
SnapshotLifecycleRestIT.testFullPolicy, this commit changes the test to find any snapshots starting
with `snap-`.

In the event that there are no snapshots (the failure case), this also exposes the full results map
so we can diagnose why a failure occurred.

Relates to elastic#50358

* Use a more imperative style for checking
@dakrone dakrone added backport :Data Management/ILM+SLM Index and Snapshot lifecycle management v7.7.0 labels Jan 24, 2020
@elasticmachine
Copy link
Collaborator

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

@dakrone dakrone changed the title [7.x] Check all snapshots in SnapshotLifecycleRestIT.testFullPolicy (#51386) [7.x] Check all snapshots in SnapshotLifecycleRestIT.testFullP… Jan 24, 2020
@dakrone dakrone merged commit 8560847 into elastic:7.x Jan 24, 2020
@dakrone dakrone deleted the backport/7.x/pr-51386 branch January 24, 2020 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :Data Management/ILM+SLM Index and Snapshot lifecycle management v7.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants