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 ism #50

Closed
wants to merge 81 commits into from
Closed

Add ism #50

wants to merge 81 commits into from

Conversation

tdegiacinto
Copy link
Contributor

Hi, in complement of #45 here is a PR to add resources for ILM.

JHeilCoveo and others added 2 commits December 20, 2019 04:25
* remove not found function from create
* Add forgotten return object
fix unit test


fix build
@phillbaker
Copy link
Owner

@tdegiacinto thanks for the PR. Just to clarify - this PR is for the opendistro index version of lifecycle management, with the resource named elasticsearch_odfe_policy, correct?

@phillbaker
Copy link
Owner

The docs for this API are here: https://opendistro.github.io/for-elasticsearch-docs/docs/ism/api/.

@tdegiacinto
Copy link
Contributor Author

Indeed this PR is for adding opendistro index lifecycle management.

@fsperling
Copy link

Is there something missing to get this merged?

@springerigor
Copy link

I would appreciate having this PR merged.

@phillbaker
Copy link
Owner

This has been squashed and cherry-picked onto master as 6e8a88f. Thanks for the effort here.

@phillbaker phillbaker closed this May 3, 2020
@springerigor
Copy link

Awesome @phillbaker 🙇‍♂️ Waiting for a released version 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants