-
Notifications
You must be signed in to change notification settings - Fork 527
Add support for Gude 8311 PDUs #865
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
base: master
Are you sure you want to change the base?
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA or my organization already has a signed CLA. |
|
Thank you for your contribution. This pull request has been marked as stale as it has not passed the automated tests and there was no activity for the last 14 days. This pull request will be closed due to inactivity after 60 days, if no action is taken. |
|
The tests in this PR and the four before it fail because of this: Can someone from the team fix this and re-trigger the checks please? |
|
Thank you for your contribution. This pull request has been marked as stale as it has not passed the automated tests and there was no activity for the last 14 days. This pull request will be closed due to inactivity after 60 days, if no action is taken. |
|
bump to avoid auto-closing |
|
Thank you for your contribution! Can you rebase your change on master? The CI should be fixed now. |
Basically a copy of the existing Gude 8310 section, changing the 27 to 62 in the OID.
This minor patch adds support for the Gude 8311 PDU model, which appears to be the successor of the 8310 and has a slightly different OID number (62 instead of 27). Checkmk already supports the 8310 out-of-the-box, but the 8311 is unsupported as of today.
The added code is a copy of the existing Gude 8310 section, changing the model name and replacing the
.27by.62in the OID (once indetect, once infetch).