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

tests: add bmpserver logging #17207

Merged
merged 3 commits into from
Oct 25, 2024
Merged

Conversation

louis-6wind
Copy link
Contributor

@louis-6wind louis-6wind commented Oct 23, 2024

Add bmpserver logging

@frrbot frrbot bot added bugfix tests Topotests, make check, etc labels Oct 23, 2024
Apply black to bmpserver

Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
bmpserver infinitely loops after the clients has closed the TCP session.
In this situation, recv() returns empty data.

Detect session close immediately.

Fixes: 875511c ("topotests: add basic bmp collector")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
Add bmpserver logging

Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
@github-actions github-actions bot added size/XXL and removed size/L labels Oct 24, 2024
@louis-6wind louis-6wind changed the title tests: add bmpserver logging tests: add bmp logging Oct 24, 2024
@github-actions github-actions bot added size/L and removed size/XXL labels Oct 24, 2024
@louis-6wind louis-6wind changed the title tests: add bmp logging tests: add bmpserver logging Oct 24, 2024
@ton31337 ton31337 merged commit 2606f84 into FRRouting:master Oct 25, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix master size/L tests Topotests, make check, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants