-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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] Add reindex warning for mapping type changes #32081
Conversation
Pinging @elastic/kibana-operations |
@@ -173,37 +173,6 @@ export class WarningsFlyoutStep extends React.Component< | |||
documentationUrl="https://www.elastic.co/guide/en/apm/get-started/master/apm-release-notes.html" | |||
/> | |||
)} | |||
|
|||
{warnings.includes(ReindexWarning.booleanFields) && ( |
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.
Doesn't apply to 7.0 so I removed this and the allField warnings.
b20a627
to
b8c4867
Compare
@gchaps Do you have any feedback on the copy here? For reference, here are the other warnings we've added in prior versions: |
💔 Build Failed |
b8c4867
to
621ff70
Compare
💚 Build Succeeded |
@joshdover Can we use the same text as in the first warning on the page: This index requires destructive changes that can't be undone Back up your index, then proceed with the reindex by accepting each breaking change. |
621ff70
to
38cfa1f
Compare
💔 Build Failed |
💚 Build Succeeded |
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, tested
Summary
This adds a warning to the reindex process that will display when the user is reindexing an index that does not use a type name of
_doc
for the 7.x -> 8.x upgrade.Test data:
https://drive.google.com/file/d/1DNdzUgeNwsWKaDwaWQOiOzEwibouqov3/view?usp=sharing