-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[errors/multi.allow_explicit_index] move error handling to browser #14184
[errors/multi.allow_explicit_index] move error handling to browser #14184
Conversation
A part of elastic#14163, this removes the portion of the healthCheck that tries to verify that rest.action.multi.allow_explicit_index is not set to false. Instead, a ui module was created that will check errors from elasticsearch for this specific scenario, and exposes a method that will display a nicer "fatal error" screen that informs the user about what they should do, and navigates away from the now broken app.
…-allow-explicit-index-handler
This is my favorite error message in all of Kibana. Seriously. We should update all gracefully handled errors (e.g. xpack auth issues for monitoring/security/reporting) with a page like this. It's slightly playful but not over the top. It explains what the problem is, why it's a problem for Kibana, and how to remedy it. cc @elastic/kibana-monitoring @elastic/kibana-sharing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
<h3>Ok, how do I fix this?</h3> | ||
<ol> | ||
<li>Remove <code>rest.action.multi.allow_explicit_index: false</code> from your Elasticsearch config file.</li> | ||
<li>Restart elasticsearch.</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
elasticsearch
-> Elasticsearch
…-allow-explicit-index-handler
…14184) * [errors/multi.allow_explicit_index] move error handling to browser A part of #14163, this removes the portion of the healthCheck that tries to verify that rest.action.multi.allow_explicit_index is not set to false. Instead, a ui module was created that will check errors from elasticsearch for this specific scenario, and exposes a method that will display a nicer "fatal error" screen that informs the user about what they should do, and navigates away from the now broken app. * [es/healthCheck] remove old test * fix typo (cherry picked from commit 2b7808b)
6.x/6.1: 375d5b6 |
…lastic#14184) * [errors/multi.allow_explicit_index] move error handling to browser A part of elastic#14163, this removes the portion of the healthCheck that tries to verify that rest.action.multi.allow_explicit_index is not set to false. Instead, a ui module was created that will check errors from elasticsearch for this specific scenario, and exposes a method that will display a nicer "fatal error" screen that informs the user about what they should do, and navigates away from the now broken app. * [es/healthCheck] remove old test * fix typo
…lastic#14184) * [errors/multi.allow_explicit_index] move error handling to browser A part of elastic#14163, this removes the portion of the healthCheck that tries to verify that rest.action.multi.allow_explicit_index is not set to false. Instead, a ui module was created that will check errors from elasticsearch for this specific scenario, and exposes a method that will display a nicer "fatal error" screen that informs the user about what they should do, and navigates away from the now broken app. * [es/healthCheck] remove old test * fix typo
A part of #14163, this removes the portion of the healthCheck that tries to verify that rest.action.multi.allow_explicit_index is not set to false. Instead, a ui module was created that will check errors from elasticsearch for this specific scenario, and exposes a method that will display a nicer "fatal error" screen that informs the user about what they should do, and navigates away from the now broken app.
Please, suggest as many edits as you like to the error page text. I'm horrible at this stuff.
To test this out run: