-
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
Fleet apache: convert status.total_kbytes to status.total_bytes #23022
Conversation
Pinging @elastic/integrations (Team:Integrations) |
@jsoriano let me assign you to this PR as it's similar to the last one in terms of the Apache integration. |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Steps errorsExpand to view the steps failures
|
Test | Results |
---|---|
Failed | 0 |
Passed | 2282 |
Skipped | 505 |
Total | 2787 |
/test |
I don't see anything here that would make this only function while under fleet? Maybe I'm missing something. |
The change is introduced in the beats/metricbeat/module/apache/status/status.go Lines 98 to 100 in ad10da0
|
Thanks! |
…tic#23022) * Fleet apache: convert status.total_kbytes to status.total_bytes * Update CHANGELOG (cherry picked from commit 13ae2e8)
Update the fields for the apache.status data stream. The fields reported from agent changed due to elastic/beats#23022.
Update the fields for the apache.status data stream. The fields reported from agent changed due to elastic/beats#23022.
What does this PR do?
This PR modifies the the
apache.status
metricset to reportapache.status.total_bytes
instead ofapache.status.total_kbytes
in the fleet mode.Why is it important?
This change is required to improve overview dashboard and support human readable format (expects bytes input).
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.How to test this PR locally
It will be tested by the CI in the Integrations project.
Related issues
Use cases