Skip to content
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

Conversation

antoninbas
Copy link
Member

  • Gracefully shutdown exporter connections before stopping collector.
  • Do not log an error for graceful client shutdown (EOF error).

@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Merging #323 (b494921) into main (a7df05f) will decrease coverage by 0.05%.
The diff coverage is 66.66%.

Additional details and impacted files

Impacted file tree graph

@@            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              
Flag Coverage Δ
integration-tests 51.50% <66.66%> (+0.11%) ⬆️
unit-tests 72.55% <66.66%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
pkg/collector/process.go 86.03% <0.00%> (ø)
pkg/collector/tcp.go 65.43% <75.00%> (+1.32%) ⬆️

... and 1 file with indirect coverage changes

* 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>
@antoninbas antoninbas force-pushed the avoid-misleading-error-messages-in-BenchmarkMultipleExportersToCollector branch from 9bf9b57 to b494921 Compare September 29, 2023 20:30
Copy link
Contributor

@heanlan heanlan left a 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.

Copy link
Contributor

@dreamtalen dreamtalen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antoninbas antoninbas merged commit a678292 into vmware:main Sep 29, 2023
7 of 9 checks passed
@antoninbas antoninbas deleted the avoid-misleading-error-messages-in-BenchmarkMultipleExportersToCollector branch September 29, 2023 22:09
dreamtalen pushed a commit to dreamtalen/go-ipfix that referenced this pull request Oct 2, 2023
* 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>
dreamtalen pushed a commit that referenced this pull request Oct 2, 2023
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants