-
Notifications
You must be signed in to change notification settings - Fork 25.3k
Correctly handling data stream settings when component templates are used #130394
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
Correctly handling data stream settings when component templates are used #130394
Conversation
Pinging @elastic/es-data-management (Team:Data Management) |
Hi @masseyke, I've created a changelog YAML for you. |
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 👍🏻
I don't think we need to label this as |
Yeah, I initially labeled it as >non-issue and then added >bug for 9.1.0... and then remembered that we haven't actually released 9.1.0 yet. Removing the label now. |
💔 Backport failed
You can use sqren/backport to manually backport by running |
…used (elastic#130394) (cherry picked from commit ae450da) # Conflicts: # server/src/main/java/org/elasticsearch/cluster/metadata/MetadataDataStreamsService.java
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
…used (elastic#130394) (cherry picked from commit ae450da) # Conflicts: # server/src/main/java/org/elasticsearch/cluster/metadata/MetadataDataStreamsService.java
If a composable template had no template structure of its own, and only used settings and mappings pulled in from component templates, then updating settings on a data stream built from that composable template could fail. For example:
This change fixes this bug by correctly (and safely) resolving all settings for a data stream for validation, rather than only the ones from the top-level composable template.