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
Disable unnecessary metrics #2147
Disable unnecessary metrics #2147
Conversation
Vert.x's client, server and verticles metrics adds a lot of allocations and their metrics exposed add little value in a kubernetes environment or our case. Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
/cc @matzew @aavarghese |
/test unit-tests_eventing-kafka-broker_main |
+1 on doing this! |
Codecov Report
@@ Coverage Diff @@
## main #2147 +/- ##
============================================
+ Coverage 66.54% 66.57% +0.02%
- Complexity 677 678 +1
============================================
Files 142 142
Lines 9141 9146 +5
Branches 196 196
============================================
+ Hits 6083 6089 +6
Misses 2652 2652
+ Partials 406 405 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
.addDisabledMetricsCategory(MetricsDomain.HTTP_CLIENT) | ||
.addDisabledMetricsCategory(MetricsDomain.HTTP_SERVER) | ||
.addDisabledMetricsCategory(MetricsDomain.VERTICLES) | ||
.addDisabledMetricsCategory(MetricsDomain.NET_CLIENT) | ||
.addDisabledMetricsCategory(MetricsDomain.NET_SERVER) | ||
.addDisabledMetricsCategory(MetricsDomain.EVENT_BUS) | ||
.addDisabledMetricsCategory(MetricsDomain.DATAGRAM_SOCKET) | ||
// NAMED_POOL allocates a lot, so disable it. |
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.
+1
we could factor those into its own method for a more compact execution - looks like the more we add, the more verbose this gets.
Just a comment 😅
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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: matzew, pierDipi The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@pierDipi what about doing backports? |
/cherry-pick release-1.4 |
/cherry-pick release-1.3 |
/cherry-pick release-1.2 |
@pierDipi: new pull request created: #2154 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@pierDipi: new pull request created: #2155 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@pierDipi: new pull request created: #2156 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Vert.x's client, server and verticles metrics adds a lot of
allocations and their metrics exposed add little value in a
kubernetes environment or in our case.
Signed-off-by: Pierangelo Di Pilato pierdipi@redhat.com
Fixes #
Proposed Changes
Release Note