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
add sensor support for cefcModuleOperStatus #15026
add sensor support for cefcModuleOperStatus #15026
Conversation
Could this be implemented in yaml instead? We try to only use php if it needs more advanced parsing |
….com/d-k-7/librenms into add-sensors-for-cefcModuleOperStatus
@Jellyfrog I implemented this in yaml rather than php. I also added the sensors to a group to help improve readability. |
This pull request has been mentioned on LibreNMS Community. There might be relevant details there: |
* add sensor support for cefcModuleOperStatus * add tests for cefcModuleOperStatus * revert sensors/state/cisco.inc.php * add sensor support for cefcModuleOperStatus using yaml * update tests for cefcModuleOperStatus * try adding tests again for cefcModuleOperStatus * include CISCO-ENTITY-FRU-CONTROL mib for cefcModuleOperStatus sensor state discovery
This PR adds state sensor support for linecard operating status for Cisco switches using the cefcModuleOperStatus OID
DO NOT DELETE THE UNDERLYING TEXT
Please note
Testers
If you would like to test this pull request then please run:
./scripts/github-apply <pr_id>
, i.e./scripts/github-apply 5926
After you are done testing, you can remove the changes with
./scripts/github-remove
. If there are schema changes, you can ask on discord how to revert.