Added Temperature sensor support for NetApp #971

Merged
merged 1 commit into from May 14, 2015

Projects

None yet

2 participants

@laf
Member
laf commented May 11, 2015

Ok a few things on this one:

Check includes/discovery/functions.inc.php. I've updated this as it looks fundamentally wrong as is. This PR was passing those values across and they were being NULL'd out due to this.

I've updated poll_sensor in includes/polling/functions.inc.php to support custom temperature includes and made use of it for this PR. This really shouldn't effect any existing temp sensors.

Screenshot.
image

Excuse the dip, this is when I was coding the PR.

@laf laf added the enhancement label May 11, 2015
@f0o f0o merged commit f19edab into librenms:master May 14, 2015

1 check failed

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