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

Rewrite Microsemi Midspan Power Sensors with mib #10959

Merged
merged 2 commits into from Dec 18, 2019

Conversation

PipoCanaja
Copy link
Contributor

@PipoCanaja PipoCanaja commented Dec 17, 2019

fixes #10952

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.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 10959
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.

@PipoCanaja PipoCanaja added the Device 🖥️ New or added device support label Dec 17, 2019
@PipoCanaja PipoCanaja self-assigned this Dec 17, 2019
@PipoCanaja
Copy link
Contributor Author

No need to notify users, the current "power" sensor is not working at all anyway.

@PipoCanaja PipoCanaja changed the title Rewrite Microsemi Midspan Power Sensors with mib - WIP Rewrite Microsemi Midspan Power Sensors with mib Dec 17, 2019
@PipoCanaja
Copy link
Contributor Author

User confirmed it works well. Can be reviewed and merged.

@PipoCanaja PipoCanaja merged commit c9ae289 into librenms:master Dec 18, 2019
@murrant
Copy link
Member

murrant commented Jan 4, 2020

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/v1-59-release-changelog-december-2019/10558/1

@lock lock bot locked as resolved and limited conversation to collaborators Mar 10, 2020
@PipoCanaja PipoCanaja deleted the midspan branch March 6, 2021 19:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Device 🖥️ New or added device support Refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add back power sensor data from Microsemi midspans
3 participants