-
Notifications
You must be signed in to change notification settings - Fork 474
[O11y][PHP-FPM] Add service.address field and update dashboard #7523
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
[O11y][PHP-FPM] Add service.address field and update dashboard #7523
Conversation
🌐 Coverage report
|
- add_fields: | ||
target: service | ||
fields: | ||
address: {{hostname}} |
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.
Why is this processor explicitly required here?
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.
@ishleenk17 The data stream uses the httpjson filebeat input in the backend. However, the httpjson filebeat input doesn't generate the service.address
field automatically that actually useful for users to filter out the details of the host. Hence we need to explicitly add this processor!
…into php-fpm-support-service-address-field Conflicts: packages/php_fpm/changelog.yml packages/php_fpm/data_stream/pool/fields/ecs.yml packages/php_fpm/data_stream/process/fields/ecs.yml packages/php_fpm/manifest.yml
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.
Looks good!
Package php_fpm - 0.6.0 containing this change is available at https://epr.elastic.co/search?package=php_fpm |
What does this PR do?
Checklist
changelog.yml
file.Related issues