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

Fix Failing Assertion in SnapshotsInProgress #37922

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

original-brownbear
Copy link
Member

@original-brownbear original-brownbear commented Jan 28, 2019


Just cleaning up random snapshot stability related issues :)

* Fix assertion by workaround for `5.6`
* Reenable test that tripped this assertion
* Closes elastic#31054
@original-brownbear original-brownbear added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v5.6.11 labels Jan 28, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@original-brownbear
Copy link
Member Author

@ywelsch thanks!

@original-brownbear original-brownbear merged commit d61e45d into elastic:5.6 Jan 30, 2019
@original-brownbear original-brownbear deleted the 31054 branch January 30, 2019 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test Issues or PRs that are addressing/adding tests v5.6.14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants