Juniper State Support #3121

Merged
merged 2 commits into from Mar 3, 2016

Projects

None yet

3 participants

@Rosiak
Contributor
Rosiak commented Feb 29, 2016

We should probably discuss the translations before merging:
I've done it this way:

unknown = unknown
empty = unknown
present = warning
ready = ok
announceOnline = ok
online = ok
anounceOffline = ok
offline = critical
diagnostic = unknown
standby = unknown

Reason for translating present to warning is that, that's the state of a PSU without power. If PSU fails state will be offline(I assume).

@Rosiak Rosiak Juniper State Support
ae37f5c
@Rosiak
Contributor
Rosiak commented Mar 1, 2016

^^

@laf
Member
laf commented Mar 2, 2016

We should just merge, it's new so can be tweaked later.

@Rosiak Rosiak Scrut fix
4bc2f5f
@laf laf merged commit 695baed into librenms:master Mar 3, 2016

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer 2 new issues
Details
@laf laf added the Discovery label Mar 3, 2016
@Rosiak Rosiak deleted the Rosiak:juniper-state branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment