-
Notifications
You must be signed in to change notification settings - Fork 734
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
Annotate APM Server Pods with co.elastic.logs/module #3840
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
💚 CLA has been signed |
jenkins test this please |
pkg/controller/apmserver/pod.go
Outdated
|
||
// DefaultAnnotations are the default annotations for the apmserver pods | ||
DefaultAnnotations = map[string]string{ | ||
annotation.FilebeatModuleAnnotation: "apmserver", |
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.
I left a message on #3721 because I'm not sure that this module is available yet, see elastic/beats#15054
Annotate APM Server pods with `co.elastic.logs/module: apmserver` annotation to give a hint to filebeat so that it uses the appropriate module to analyze the logs of the container. Signed-off-by: prateekpandey14 <[email protected]>
8bb87b3
to
156eb58
Compare
I'm closing this one because I don't think a filebeat module for the APM Server is available yet. |
Annotate APM Server pods with
co.elastic.logs/module: apm-server
annotation to give a hint to filebeat so that it uses theappropriate module to analyze the logs of the container.
fixes: #3721
Signed-off-by: prateekpandey14 [email protected]