-
Notifications
You must be signed in to change notification settings - Fork 25.3k
System index deprecation warning is not critical #79633
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
System index deprecation warning is not critical #79633
Conversation
Pinging @elastic/es-core-infra (Team:Core/Infra) |
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.
Thanks for making this change, Will! I think this is an improvement. At the same time, these logs will still show up in Upgrade Assistant due to Kibana accessing these system indices. Elasticsearch will complain to the user about something that Kibana is doing. It makes it look like our products don't play well together. What kind of impression will that leave with our users?
Also, as you pointed out, these logs aren't actionable -- so what's the point? They become noise in the upgrade process. Can we remove them entirely in 7.16?
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.
Code changes LGTM, pending a decision on the points CJ raised.
@cjcenizal Was this question something we needed to discuss before merging, or do we have other issues covering this point of discussion? |
@williamrandolph No not a a blocker. @sebelga is now looking into this problem from the Kibana/UA side and will re-raise this concern in another context if necessary. Thanks for asking! |
When we planned to fully cut off access to system indices in 8.0, we needed our system index deprecation warning to be critical. Now that we are merely hiding system indices, avoiding system index access is no longer critical, and there is very little that users can do to change their access patterns. Therefore, we should log the warning at WARN level, rather than at CRITICAL.
When we planned to fully cut off access to system indices in 8.0, we needed our system index deprecation warning to be critical. Now that we are merely hiding system indices, avoiding system index access is no longer critical, and there is very little that users can do to change their access patterns. Therefore, we should log the warning at WARN level, rather than at CRITICAL.
* System index deprecation warning is not critical (#79633) When we planned to fully cut off access to system indices in 8.0, we needed our system index deprecation warning to be critical. Now that we are merely hiding system indices, avoiding system index access is no longer critical, and there is very little that users can do to change their access patterns. Therefore, we should log the warning at WARN level, rather than at CRITICAL. * Restore production code changes
) * System index deprecation warning is not critical (#79633) When we planned to fully cut off access to system indices in 8.0, we needed our system index deprecation warning to be critical. Now that we are merely hiding system indices, avoiding system index access is no longer critical, and there is very little that users can do to change their access patterns. Therefore, we should log the warning at WARN level, rather than at CRITICAL. * Restore production code changes
When we planned to fully cut off access to system indices in 8.0, we needed our system index deprecation warning to be critical. Now that we are merely hiding system indices, avoiding system index access is no longer critical, and there is very little that users can do to change their access patterns. Therefore, we should log the warning at
WARN
level, rather than atCRITICAL
.