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 protocol stats to the network plugin #382

Merged
merged 2 commits into from
Dec 4, 2015
Merged

Add protocol stats to the network plugin #382

merged 2 commits into from
Dec 4, 2015

Conversation

nathanielc
Copy link
Contributor

This add the system wide network protocol stats to the net plugin.

Uses a new method in gopsutil see shirou/gopsutil#109

Basically exposes all the data in /proc/net/snmp as

net_PROTOCOL_COLNAME

for example:

Excerpt from /proc/net/snmp

Udp: InDatagrams NoPorts InErrors OutDatagrams RcvbufErrors SndbufErrors InCsumErrors IgnoredMulti
Udp: 67688650 3723361 58108572 124599763 58108572 0 0 318767

adds measurement net_udp_rcvbuferrors with value 58108572 etc.

@@ -42,6 +42,7 @@ same type.
- [#364](https://github.com/influxdb/telegraf/pull/364): Support InfluxDB UDP output.
- [#370](https://github.com/influxdb/telegraf/pull/370): Support specifying multiple outputs, as lists.
- [#372](https://github.com/influxdb/telegraf/pull/372): Remove gosigar and update go-dockerclient for FreeBSD support. Thanks @MerlinDMC!
- [#382](https://github.com/influxdb/telegraf/pull/382): Add system wide network protocol stats to `net` plugin.
Copy link
Contributor

Choose a reason for hiding this comment

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

this should go higher up, under 0.2.3

@sparrc
Copy link
Contributor

sparrc commented Nov 20, 2015

looks good, thanks @nathanielc! will get this merged once the gopsutil PR is merged

@sparrc sparrc merged commit c83f220 into master Dec 4, 2015
@sparrc sparrc deleted the nc-net-protos branch December 7, 2015 19:03
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.

2 participants