refactor: Move mib based polling into yaml config files #5234

Merged
merged 3 commits into from Jan 3, 2017

Projects

None yet

4 participants

@laf
Member
laf commented Dec 24, 2016

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.

Moved mib polling calls into discover_device() function so they only need defining in the yaml definitions file now.

Updated docs to show new examples.

Updated html page to tell users they can enable per device + have per device check to show mib page.

Move code from includes/discovery/os/*.inc.php to includes/definitions/*.yaml.

@laf laf refactor: More mib based polling into yaml config files
154d33c
@laf laf changed the title from refactor: More mib based polling into yaml config files to refactor: Move mib based polling into yaml config files Dec 29, 2016
@laf
Member
laf commented Dec 29, 2016

bump

doc/Extensions/MIB-based-polling.md
### Discovery
-You need to add your desired MIBs to `/opt/librenms/mibs` folder. Afterwards you need to register your MIBs to the discovery function.
+You need to add your desired MIBs to `/opt/librenms/mibs` folder. Afterwards you need to register your MIBs to the OS definition.
@murrant
murrant Jan 3, 2017 edited Contributor

This better?

You need to add your desired MIBs to `/opt/librenms/mibs` folder. 
You will then need to create a list of the OIDs you wish to use in the OS definition
@murrant
Contributor
murrant commented Jan 3, 2017

Other than the small documentation change +1

@laf laf doc update
31d1ffe
@laf
Member
laf commented Jan 3, 2017

Doc updated.

@scrutinizer-notifier

The inspection completed: 658 Issues, 18 Patches

@murrant murrant Update MIB-based-polling.md
5a5b8e6
@murrant murrant merged commit 7f5c555 into librenms:master Jan 3, 2017

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:mib-polling-move branch Jan 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment