fix: APC PDU2 Voltage Discovery #5276

Merged
merged 1 commit into from Jan 1, 2017

Projects

None yet

4 participants

@Rosiak
Contributor
Rosiak commented Jan 1, 2017

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

I were seeing issues with the "pre-calculated" high/low warn/limit on the voltage part on a PDU.
Quickly ran disco debug and saw this issue.

rPDU2 capable devices respond to rPDU table, hence this can cause issues.

Tested on APC AP8858EU3

@Rosiak Rosiak fix: APC PDU2 Voltage Discovery
8f3c1e4
@scrutinizer-notifier

The inspection completed: 1 new issues

@laf laf merged commit 4904f0e into librenms:master Jan 1, 2017

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf added a commit that referenced this pull request Jan 1, 2017
@Rosiak @laf Rosiak + laf fix: APC PDU2 Voltage Discovery (#5276) bb23fb5
@Rosiak Rosiak deleted the Rosiak:apc-pdu2-voltage-fix branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment