-
Notifications
You must be signed in to change notification settings - Fork 8.4k
Revert "[Fleet] Improve validation for dynamic Kafka topics" #219415
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
Conversation
This reverts commit 5903c7a.
Pinging @elastic/fleet (Team:Fleet) |
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.
🚀 That feature was never available to user right? user cannot create outputs in serverless and 9.1 is not yet released
Correct this feature was only ever in 9.1.0. |
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 for the docs. 👍
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
cc @kpollich |
…#219415) Reverts elastic#212422 We don't want to surface this Beats-specific syntax for dynamic topics because it greatly harms our ability to migrate users to OTel. The recommended way to handle complex multi-field interpolations for topic names is to use an `add_field` Beats processor to add the desired topic name as a distinct field, and then use that field for the dynamic topic name in the Fleet-managed Kafka output. I'll be filing a docs issue to get this in our public docs as well.
Reverts #212422
We don't want to surface this Beats-specific syntax for dynamic topics because it greatly harms our ability to migrate users to OTel.
The recommended way to handle complex multi-field interpolations for topic names is to use an
add_field
Beats processor to add the desired topic name as a distinct field, and then use that field for the dynamic topic name in the Fleet-managed Kafka output. I'll be filing a docs issue to get this in our public docs as well.