-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
7.x Index management compatibility with ES 8.x #122933
Comments
Pinging @elastic/platform-deployment-management (Team:Deployment Management) |
I'd consider this a bug but not a blocker, because the user can still use Console to access the underlying ES APIs and perform operations on indices and index templates. So we can ship a fix in a 7.17.patch release. See #122459 (comment) for a reference to how folks are solving this problem elsewhere. |
I am not sure I follow on this. Why is this a bug? We are using a 7.x API in https://www.elastic.co/guide/en/elasticsearch/reference/current/removal-of-types.html |
@sebelga The related issue #122459 has more context, but the summary is that during the upgrade process to 8, some or all ES nodes will be on 8.x while Kibana is still on 7.last. We want Kibana to continue to function in this situation. So we need to ensure any requests from Kibana 7.last won't fail if ES is on 8.x. |
Thanks @cjcenizal. I haven't been part of the decision process on this but it does seem that supporting this scenario is adding a lot of potential headaches and I am not sure I see the benefit (but again maybe I am missing something). From what we have been doing in the stack management team during the 7.x cycle this was never contemplated. |
@sebelga Right. This is a very late-breaking requirement. |
Let's pause any work on this until we've made a Kibana-wide decision on how to address these types of bugs. |
I'm going to close this since it seems unlikely we'll be able to act on it and it seems like a corner case. Let's reopen it if it comes up as a real problem. |
Index management uses the deprecated
include_type_name
parameter in a couple of places:https://github.com/elastic/kibana/blob/7.17/x-pack/plugins/index_management/server/routes/api/mapping/register_mapping_route.ts#L33
https://github.com/elastic/kibana/blob/7.17/x-pack/plugins/index_management/server/routes/api/templates/register_get_routes.ts#L30
https://github.com/elastic/kibana/blob/7.17/x-pack/plugins/upgrade_assistant/server/lib/reindexing/reindex_actions.ts#L223
I didn't test the code, but it's possible that these API calls would fail if Kibana is connected to Elasticsearch 8.x
Related to #122459
The text was updated successfully, but these errors were encountered: