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

Add wait_for_health=yellow to reindex snippets #18295

Merged
merged 1 commit into from
May 12, 2016

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented May 12, 2016

This should help the tests pass more consistently. Should.

This also removes from ?pretty from the docs. It isn't a thing
with // CONSOLE.

Relates to
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+multijob-os-compatibility/os=oraclelinux/399/console

This should help the tests pass more consistently. Should.

This also removes from `?pretty` from the docs. It isn't a thing
with `// CONSOLE`.

Relates to
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+multijob-os-compatibility/os=oraclelinux/399/console
@nik9000 nik9000 added >test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI :Reindex API v5.0.0-alpha3 labels May 12, 2016
@nik9000
Copy link
Member Author

nik9000 commented May 12, 2016

@ywelsch, this is my reaction to the failed build you sent me.

@ywelsch
Copy link
Contributor

ywelsch commented May 12, 2016

LGTM (in the future, we might not need these wait_for_health=yellow, see #9126)

@nik9000
Copy link
Member Author

nik9000 commented May 12, 2016

Oh cool! I'm glad we are working on that!

@nik9000 nik9000 merged commit 94317b1 into elastic:master May 12, 2016
@lcawl lcawl added :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. and removed :Reindex API labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI v5.0.0-alpha3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants