Skip to content
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

IP Infusion OcNOS basic detection #14588

Merged
merged 3 commits into from Nov 7, 2022
Merged

IP Infusion OcNOS basic detection #14588

merged 3 commits into from Nov 7, 2022

Conversation

murrant
Copy link
Member

@murrant murrant commented Nov 7, 2022

Please give a short description what your pull request is for

DO NOT DELETE THE UNDERLYING TEXT

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

  • Have you followed our code guidelines?
  • If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.
  • If my Pull Request makes discovery/polling/yaml changes, I have added/updated test data.

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.

@murrant murrant added the Device 🖥️ New or added device support label Nov 7, 2022
@murrant murrant merged commit 9d3d81b into librenms:master Nov 7, 2022
@murrant murrant deleted the ocnos branch November 7, 2022 19:09
@kvedder-amplex
Copy link
Contributor

Thanks @murrant. I had written a module for OcNOS and was using it locally, even better to get it into the community.

@murrant
Copy link
Member Author

murrant commented Nov 16, 2022

@kvedder-amplex if you have any modules/sensor additions for it, send them up :) I didn't get a chance to look at the mibs even yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Device 🖥️ New or added device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants