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

[FlowAggregator] Close connection to IPFIX collector on Stop #6635

Conversation

antoninbas
Copy link
Contributor

When the IPFIX exporter is stopped in the Flow Aggregator, the connection to the collector should be explicitly closed.

When the IPFIX exporter is stopped in the Flow Aggregator, the
connection to the collector should be explicitly closed.

Co-authored-by: Yun-Tang Hsu <[email protected]>

Signed-off-by: Antonin Bas <[email protected]>
@antoninbas
Copy link
Contributor Author

Thanks @yuntanghsu for pointing this out!

@antoninbas antoninbas requested review from tnqn and yuntanghsu August 27, 2024 22:03
Copy link
Contributor

@yuntanghsu yuntanghsu 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 added action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes. labels Aug 28, 2024
@antoninbas
Copy link
Contributor Author

/test-all

@antoninbas antoninbas merged commit 8167f95 into antrea-io:main Aug 28, 2024
52 of 59 checks passed
@antoninbas antoninbas deleted the fa-close-connection-to-collector-when-ipfix-exporter-is-stopped branch August 28, 2024 21:43
antoninbas added a commit to antoninbas/antrea that referenced this pull request Aug 28, 2024
When the IPFIX exporter is stopped in the Flow Aggregator, the
connection to the collector should be explicitly closed.

Signed-off-by: Antonin Bas <[email protected]>
antoninbas added a commit that referenced this pull request Sep 3, 2024
When the IPFIX exporter is stopped in the Flow Aggregator, the
connection to the collector should be explicitly closed.

Signed-off-by: Antonin Bas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants