Skip to content

[UA] Add reindex warning for mapping type changes - #32081

Merged
joshdover merged 2 commits into
elastic:7.xfrom
joshdover:ua-typename-warning
Mar 13, 2019
Merged

joshdover merged 2 commits into
elastic:7.xfrom
joshdover:ua-typename-warning

Conversation

@joshdover

Copy link
Copy Markdown
Contributor

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.

image

Test data:
https://drive.google.com/file/d/1DNdzUgeNwsWKaDwaWQOiOzEwibouqov3/view?usp=sharing

@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations

@joshdover joshdover changed the title Add reindex warning for mapping type changes [UA] Add reindex warning for mapping type changes Feb 26, 2019

Copy link
Copy Markdown
Contributor Author

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.

@joshdover

Copy link
Copy Markdown
Contributor Author

@gchaps Do you have any feedback on the copy here? For reference, here are the other warnings we've added in prior versions:

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@gchaps

gchaps commented Feb 27, 2019

Copy link
Copy Markdown
Contributor

@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.

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@tylersmalley tylersmalley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants