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

Default retries for ISM Actions #62

Closed
adityaj1107 opened this issue Jun 3, 2021 · 0 comments
Closed

Default retries for ISM Actions #62

adityaj1107 opened this issue Jun 3, 2021 · 0 comments
Labels
enhancement New request

Comments

@adityaj1107
Copy link
Contributor

Issue by dbbaughe
Friday Apr 02, 2021 at 16:28 GMT
Originally opened as opendistro-for-elasticsearch/index-management#430


We have noticed sometimes users aren't fully aware there are options like retries for when an action fails and error notifications to be notified about it. Usually not the worst thing in the world, but when it comes to things like rollover, if this enters into a failed state and they do not have those configured an index can very quickly grow too large and cause cluster instabilities.

One potential solution for the transient errors at least is to have ISM set default retry configurations instead of none and the user would have to explicitly set the retry to 0 if they do not want it to be retried. This would protect against the transient failures for actions that could have consequences for not executing.

Another solution (or can both be done) is just a better UI to show what is allowed. Right now the JSON editor is not the most user friendly so it is probably hard for people to realize they can set these extra options. If we had an updated UI that had explicit areas for configuring retries and error notifications along with help text explaining why they should use it and a warning for some of these actions like rollover if they are not configured then this could also solve the issue. i.e. while configuring a rollup action on the UI we have a yellow warning box that says something like "If retries or not configured for rollover it's possible to fail and have the index grow too large" along with some warnings about lack of error notifications.

Will keep this open to get some other thoughts on it.

@adityaj1107 adityaj1107 added the enhancement New request label Jun 3, 2021
thalurur pushed a commit to thalurur/open-index-management that referenced this issue Oct 22, 2021
* Added PolicyDetails page

* Code cleanup, simplified page states, fixed error handling

Signed-off-by: Eric Lobdell <lobdelle@amazon.com>

* Fixed duplicate import, removed outdated test, updated policy detail test

Signed-off-by: Eric Lobdell <lobdelle@amazon.com>
@bowenlan-amzn bowenlan-amzn added this to Small tasks in Index Management Roadmap Feb 28, 2022
Index Management Roadmap automation moved this from Small tasks to Done Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New request
Development

No branches or pull requests

2 participants