Graph multi-lane optics on Juniper equipment #6377

Merged
merged 6 commits into from Apr 13, 2017

Conversation

Projects
None yet
5 participants
@remydb
Contributor

remydb commented Apr 7, 2017

Add support for graphing optics with multiple lanes (QSFP+, QSFP28, CFP) on Juniper equipment.
Fixes #4019

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.

  • Have you signed the Contributors agreement - please do NOT submit a pull request unless you have (signing the agreement in the same pull request is fine). Your commit message for signing the agreement must appear as per the docs.
  • Have you followed our code guidelines?

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926

@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Apr 7, 2017

Thank you for submitting a PR @remydb! We have found the following @laf based on the history of these files to review this PR.

Thank you for submitting a PR @remydb! We have found the following @laf based on the history of these files to review this PR.

@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

scrutinizer-notifier Apr 13, 2017

The inspection completed: No new issues

The inspection completed: No new issues

@laf laf merged commit 3884a3c into librenms:master Apr 13, 2017

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@remydb remydb referenced this pull request Apr 19, 2017

Merged

Fix duplicate dBm readings for single-lane optics #6460

2 of 2 tasks complete

@librenms librenms locked as resolved and limited conversation to collaborators May 19, 2018

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