-
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
Cherry-pick #21197 to 7.x: Fix libbeat.output.*.bytes metrics for ES #21210
Conversation
…1197) ## What does this PR do? This PR passes a missing parameter to the constructor `elasticsearch.Client`. ## Why is it important? The missing parameter prevented Beats from reporting the metrics `libbeat.output.write.bytes` and `libbeat.output.read.bytes` in case of Elasticsearch output. ## Related issues Closes elastic#20752 (cherry picked from commit 78dc0cf)
Pinging @elastic/integrations-services (Team:Services) |
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
Cherry-pick of PR #21197 to 7.x branch. Original message:
What does this PR do?
This PR passes a missing parameter to the constructor
elasticsearch.Client
.Why is it important?
The missing parameter prevented Beats from reporting the metrics
libbeat.output.write.bytes
andlibbeat.output.read.bytes
in case of Elasticsearch output.Checklist
- [ ] I have made corresponding changes to the documentation- [ ] I have made corresponding change to the default configuration filesCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.How to test this PR locally
Example configuration:
Read events from
test.log
. Then visit http://localhost:5066/stats?pretty and check that the aforementioned counters are greater than zero.Related issues
Closes #20752