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

Add collector for netstats tcp packet counters for FreeBSD. #3177

Merged
merged 2 commits into from
Feb 2, 2025

Conversation

Rin0913
Copy link
Contributor

@Rin0913 Rin0913 commented Nov 10, 2024

I would like to extend network support for Prometheus node exporter on FreeBSD.

In this PR, I've added two counters to log the total TCP packets sent and received.

I hope everyone can discuss together whether this approach is feasible.

@Rin0913 Rin0913 force-pushed the master branch 7 times, most recently from 20a4ba5 to 4b51602 Compare November 10, 2024 02:11
collector/netstat_freebsd.go Outdated Show resolved Hide resolved
collector/netstat_freebsd.go Outdated Show resolved Hide resolved
collector/netstat_freebsd_test.go Outdated Show resolved Hide resolved
collector/netstat_freebsd.go Outdated Show resolved Hide resolved
Rin0913

This comment was marked as duplicate.

Copy link
Contributor Author

@Rin0913 Rin0913 left a comment

Choose a reason for hiding this comment

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

Fixed.

@Rin0913
Copy link
Contributor Author

Rin0913 commented Nov 14, 2024

@SuperQ
Hello, is there anything else I should work on?

collector/netstat_freebsd.go Outdated Show resolved Hide resolved
@Rin0913
Copy link
Contributor Author

Rin0913 commented Jan 16, 2025

@SuperQ

Hello, could you please merge this PR? Or there is anything I should work on?
Thank you.

@discordianfish discordianfish merged commit 43dfdce into prometheus:master Feb 2, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants