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

UDP Socket Memory Limit hit / Overflow #55

Closed
bpereto opened this issue Nov 22, 2021 · 7 comments
Closed

UDP Socket Memory Limit hit / Overflow #55

bpereto opened this issue Nov 22, 2021 · 7 comments
Labels
bug Something isn't working transport Related to transport libraries (Kafka)

Comments

@bpereto
Copy link

bpereto commented Nov 22, 2021

I think there is some issue introduced in updating sarama to 1.30.0 or its deps (#47) as we see UDP Socket Memory limit hit / overflow.

The working commit is: 17a96d9
The Overflow surfaces with: ec08b78

OK: netdata graph with commit 17a96d9
Screenshot 2021-11-22 at 12 10 43

NOK: netdata graph with commit ec08b78
Screenshot 2021-11-22 at 12 11 23

@shyam334
Copy link
Contributor

To isolate the fault, wondering if you have tried the commit/PR just before ec08b78, 7baa828. (where sarama was updated, but not go version)

@bpereto
Copy link
Author

bpereto commented Nov 29, 2021

yes, I verified the commit 7baa828 with the same result.

@lspgn
Copy link
Member

lspgn commented Nov 29, 2021

Thank you for reporting this issue, are you able to provide more collected metrics (eg: Prometheus endpoint)?
Does this surface when using a file transport? (assuming no, but would like to know before ruling this out)
Are you able to get the count of messages produced into Kafka with both versions?
Is it NetFlow/IPFIX or sFlow?

@lspgn
Copy link
Member

lspgn commented Dec 20, 2021

Hello,
Is this still happening?

@bpereto
Copy link
Author

bpereto commented Jan 25, 2022

Hi, sorry for the delay, I will test again.
The change is the update to sarama 1.30.1 in right? 05b4362

the release 1.30.1 of sarama has a warning, which I don't know if this applies to goflow?
Screenshot 2022-01-25 at 15 39 00

regression for throughput drop:
IBM/sarama#2071

I will test again with the newest commits of goflow.

@bpereto
Copy link
Author

bpereto commented Jan 31, 2022

The performance issue is resolved with the latest commit of goflow (8d59905) and latest sarama release 1.31.0, probably a regression in sarama.

Thank you for reporting this issue, are you able to provide more collected metrics (eg: Prometheus endpoint)?

I think this is now obsolete.

Does this surface when using a file transport? (assuming no, but would like to know before ruling this out)

No, writing directly to a file (tmpfs) does not show a problem.

Is it NetFlow/IPFIX or sFlow?

It's ipfix (v10).

@lspgn lspgn added bug Something isn't working transport Related to transport libraries (Kafka) labels Feb 2, 2022
@lspgn
Copy link
Member

lspgn commented Feb 2, 2022

Thank you for confirming. Will close the Issue.

@lspgn lspgn closed this as completed Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working transport Related to transport libraries (Kafka)
Projects
None yet
Development

No branches or pull requests

3 participants