-
Notifications
You must be signed in to change notification settings - Fork 24.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
Fix broken link for grok patterns in documentation #78878
Fix broken link for grok patterns in documentation #78878
Conversation
The current link for grok patterns does not exist anymore: https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/grok-patterns I have replaced it with: https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/ecs-v1/grok-patterns But I'm unsure if it is the right link as there is also a 'legacy' folder.
Pinging @elastic/es-docs (Team:Docs) |
@nik9000, should we link to the ecs-1 pattern or the legacy pattern? I see differences in the patterns, but am unsure which one we should link to. |
@danhermann will know more. It looks like the default is the legacy patterns but I don't know if that's what the link should be. |
@nik9000, thanks for the ping. The default pattern set remains the legacy patterns. @lockewritesdocs, I'll leave it up to you whether you want to mention the availability of ECS-compliant patterns in addition to the legacy patterns. |
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. Thanks for fixing the link, @lockewritesdocs!
@elasticmachine test this please |
* Fix broken link for grok patterns in documentation The current link for grok patterns does not exist anymore: https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/grok-patterns I have replaced it with: https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/ecs-v1/grok-patterns But I'm unsure if it is the right link as there is also a 'legacy' folder. * Update link and add section for ECS migration Co-authored-by: Adam Locke <[email protected]>
💚 Backport successful
|
Thanks for highlighting this broken link @stephane-deraco! |
* Fix broken link for grok patterns in documentation The current link for grok patterns does not exist anymore: https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/grok-patterns I have replaced it with: https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/ecs-v1/grok-patterns But I'm unsure if it is the right link as there is also a 'legacy' folder. * Update link and add section for ECS migration Co-authored-by: Adam Locke <[email protected]> Co-authored-by: Stéphane DERACO <[email protected]>
The current link for grok patterns does not exist anymore:
https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/grok-patterns
I have replaced it with:
https://github.com/elastic/elasticsearch/blob/master/libs/grok/src/main/resources/patterns/ecs-v1/grok-patterns
But I'm unsure if it is the right link as there is also a 'legacy' folder.
--
Related to #76885