-
Notifications
You must be signed in to change notification settings - Fork 27
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
Avoid error logs in BenchmarkMultipleExportersToCollector #323
Avoid error logs in BenchmarkMultipleExportersToCollector #323
Conversation
antoninbas
commented
Sep 29, 2023
- Gracefully shutdown exporter connections before stopping collector.
- Do not log an error for graceful client shutdown (EOF error).
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #323 +/- ##
==========================================
- Coverage 73.53% 73.48% -0.05%
==========================================
Files 18 18
Lines 2792 2795 +3
==========================================
+ Hits 2053 2054 +1
- Misses 572 574 +2
Partials 167 167
Flags with carried forward coverage won't be shown. Click here to find out more.
|
* Gracefully shutdown exporter connections before stopping collector. * Do not log an error for graceful client shutdown (EOF error). Signed-off-by: Antonin Bas <abas@vmware.com>
9bf9b57
to
b494921
Compare
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
About the previous integration test flakiness, I think there could be a bug in the throughput calculation logic. I'm checking on 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.
LGTM
* Gracefully shutdown exporter connections before stopping collector. * Do not log an error for graceful client shutdown (EOF error). Signed-off-by: Antonin Bas <abas@vmware.com>
* Gracefully shutdown exporter connections before stopping collector. * Do not log an error for graceful client shutdown (EOF error). Signed-off-by: Antonin Bas <abas@vmware.com>