sensors: Report sensor readings by descriptive labels #1239

Merged
merged 1 commit into from Apr 22, 2016

Projects

None yet

2 participants

@fetzerch
Contributor

Adds an option UseLabels to configure how sensor readings are reported. The default reports readings using the sensor name (e.g. "in0"). With this option set to true, the readings are reported using the label (e.g. "VCore").

A similar patch has been sent to the mailing list (by Jakob Haufe sur5r@sur5r.net in 02/2010) but never got merged.
This version adds the UseLabels configuration option as suggested in the review.

@mfournier mfournier added the Feature label Sep 2, 2015
@fetzerch fetzerch sensors: Report sensor readings by descriptive labels
Adds an option UseLabels to configure how sensor readings are
reported. The default reports readings using the sensor name (e.g.
"in0"). With this option set to true, the readings are reported using
the label (e.g. "VCore").
8ed3735
@mfournier mfournier modified the milestone: Features Jan 21, 2016
@fetzerch
Contributor

I'd really like to see this upstream. Is there anything missing from my side?

@mfournier mfournier merged commit 8ed3735 into collectd:master Apr 22, 2016

1 check passed

pull-requests-github_trigger-aggregation Build #150 succeeded in 21 min
Details
@mfournier
Contributor

Thanks a lot @fetzerch ! Your patch was merged in a couple of minutes ago and will be part of version 5.6.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment