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

fix issue-1559 #1560

Merged
merged 1 commit into from Jul 29, 2015

Conversation

Projects
None yet
3 participants
@Rosiak
Contributor

Rosiak commented Jul 29, 2015

This fixes the problem.

The issue I was seeing was that rPDULoadBankConfigIndex was reporting 2 banks instead of 3(Bank 1, Bank 2, Bank Total). For that reason I was only seeing data for Bank 1 && Bank Total.

I've tested this fix on:
AP7822 B2
AP8853 02
AP8858 02

No issues here.

@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@LibreNMS-CI

LibreNMS-CI commented Jul 29, 2015

Auto-Deploy finished, Test PR at http://1560.ci.librenms.org or https://1560.ci.librenms.org

f0o added a commit that referenced this pull request Jul 29, 2015

@f0o f0o merged commit 1ad21e0 into librenms:master Jul 29, 2015

2 checks passed

Auto-Deploy Build finished.
Details
Scrutinizer No new issues
Details

@Rosiak Rosiak deleted the Rosiak:issue-1559 branch Feb 5, 2017

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