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

Remove support for legacy imports from the UI #103921

Closed
3 tasks done
Tracked by #84380
joshdover opened this issue Jun 30, 2021 · 3 comments
Closed
3 tasks done
Tracked by #84380

Remove support for legacy imports from the UI #103921

joshdover opened this issue Jun 30, 2021 · 3 comments
Labels
Breaking Change required-for-8.0 This work is required to be done before 8.0 lands, bc it relates to a breaking change or similar. Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc technical debt Improvement of the software architecture and operational architecture

Comments

@joshdover
Copy link
Contributor

joshdover commented Jun 30, 2021

Related to #41439

In Kibana 8.0, we will no longer support the legacy export format from Kibana 6.x. We've had a warning in place in the UI since 7.0 that these exports will no longer be supported.

This is issue is to scope out removal of the UI code for handling legacy imports. This code leverages the SavedObjectLoader pattern for saving these objects to the backend. These loaders can now be removed once we stop supporting legacy exports (#67607).

This should probably happen in a multiple phases:

@joshdover joshdover added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc technical debt Improvement of the software architecture and operational architecture Breaking Change required-for-8.0 This work is required to be done before 8.0 lands, bc it relates to a breaking change or similar. labels Jun 30, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@rudolf rudolf changed the title Remove support for legacy exports Remove support for legacy imports from the UI Sep 3, 2021
@jportner
Copy link
Contributor

jportner commented Sep 9, 2021

@TinaHeiligers does this issue include removing the legacy import HTTP route? The description makes it sound like we are only making UI changes. Are we keeping that route around??

Edit: I see that server side work is tracked in #41439, and it appears from that issue description that the API will be sticking around. (for others reading this comment who may be unfamiliar, "legacy import" == "dashboard import")

Edit 2: based on #41439 (comment) it sounds like we are actually going to start collecting usage data in 7.16 but go ahead and remove this API in 8.0, with the understanding that we may need to add it again if we get enough demand for it

@pgayvallet
Copy link
Contributor

does this issue include removing the legacy import HTTP route?

Nope, the legacy (dashboard) import and the legacy import (format) are two distinct things. This issue is about removing support of the legacy import format from SOM, and #41439 is about the legacy dashboard import.

it sounds like we are actually going to start collecting usage data in 7.16 but go ahead and remove this API in 8.0, with the understanding that we may need to add it again if we get enough demand for it

That's correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change required-for-8.0 This work is required to be done before 8.0 lands, bc it relates to a breaking change or similar. Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

No branches or pull requests

5 participants