-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Make file system metadata values strings #36697
Conversation
To better align with ECS and better store the values in Elasticsearch.
Possible to backport this to |
@jlind23 it's technically a breaking change, do you think we can backport it to 8.10 anyway? It would fix some things for integration developers. |
@rdner No we shouldn't backport any breaking change in a patch release. 8.11 is lined up to be out soon so better stick with 8.11. Moreover if it's a breaking change, we need to have a proper entry in the release note for this. |
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.
To better align with ECS and better store the values in Elasticsearch.
To better align with ECS and better store the values in Elasticsearch.
Checklist
- [ ] I have commented my code, particularly in hard-to-understand areas- [ ] I have made corresponding changes to the documentation- [ ] I have made corresponding change to the default configuration filesCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.How to test this PR locally
See steps in #36065
Related issues