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

[UA] Reindex searchable snapshot indices #200945

Open
jloleysens opened this issue Nov 20, 2024 · 6 comments
Open

[UA] Reindex searchable snapshot indices #200945

jloleysens opened this issue Nov 20, 2024 · 6 comments
Assignees
Labels
blocked Feature:Upgrade Assistant Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0 v9.0.0

Comments

@jloleysens
Copy link
Contributor

We need to make sure that the reindexing logic in upgrade assistant handles reindexing of indices in searchable snapshots correctly.

(More details to come as this is still under investigation)

@jloleysens jloleysens self-assigned this Nov 20, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Nov 20, 2024
@jloleysens jloleysens added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc and removed needs-team Issues missing a team label labels Nov 20, 2024
@elasticmachine
Copy link
Contributor

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

@Bamieh Bamieh added the v8.18.0 label Nov 27, 2024
@sophiec20
Copy link
Contributor

pending clarification from elasticsearch team regarding how to handle searchable snapshots during the upgrade @bytebilly

@bytebilly
Copy link
Contributor

The current recommendation is to not reindex searchable snapshots, since they're read-only by design and they can be supported using the new N-2 approach.

@jloleysens
Copy link
Contributor Author

jloleysens commented Dec 11, 2024

@sophiec20 @bytebilly , thanks for the responses! If this is the recommendation, from what I can tell there are a few options:

(1) We should not list these as "CRITICAL" deprecations, ideally this non critical status would come directly from GET /_migration/deprecations... Not much changes for UA, we just need to add some logic NOT to offer to reindex these indices.

(2) Do we want to prompt users to consider this from within UA at all? Perhaps it can be removed entirely from GET /_migration/deprecations (zero work for UA)

(3) We do want to surface the option to reindex snapshots in some way... Given time constraints this likely means some kind of documentation where we guide users through unmounting, reindexing (possibly through UA), then mounting again... This would not be a great experience, but would be possible.

(4) ...something else

@bytebilly
Copy link
Contributor

I don't have strong opinions on whether this should be handled in Kibana or on the ES API, the user experience will be the same. I'm slightly in favor of ES API, but please check with engineers and agree on implementation details.

@jloleysens
Copy link
Contributor Author

I spoke with @dakrone who said they will be going with option (2).

removed entirely from GET /_migration/deprecations (zero work for UA)

I'll close this once that work is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Feature:Upgrade Assistant Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0 v9.0.0
Projects
None yet
Development

No branches or pull requests

5 participants