Update IOS-XR watts to be dbm sensor as well #1661

Merged
merged 2 commits into from Aug 11, 2015

Projects

None yet

4 participants

@mmarchand
Contributor

Add dbm graphs for IOS-XR transceivers

Fix #1641

@mmarchand mmarchand IOS-XR does not provide dbm values for transceivers so we calculate t…
…hem from mWatts entries

Fix #1641
011181c
@laf
Member
laf commented Aug 11, 2015

All looks good to me.

We've updated our contributors agreement recently so you will need to submit a commit (in this PR is fine), just add a space next to your name and use the correct message in the commit.

Once done, we can merge in.

@Rosiak
Contributor
Rosiak commented Aug 11, 2015

What about this @laf ?
""SNMPv2-MIB$mib"" in the "includes/polling/functions.inc.php"?

@laf laf changed the title from Fix for Issue 1641 to Update IOS-XR watts to be dbm sensor as well Aug 11, 2015
@laf
Member
laf commented Aug 11, 2015

@Rosiak

That's not much of an issue, it's from code earlier to use an actual MIB which normally the sensors don't use.

@mmarchand mmarchand I agree to the conditions of the Contributor Agreement contained in d…
…oc/General/Contributing.md
f260fcc
@laf laf merged commit f266aa0 into librenms:master Aug 11, 2015

2 checks passed

Auto-Deploy Build finished.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment