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
device: Added additional sensors support for HP ILO4 #7053
Conversation
Added HP ILO4 temperature (HDD, motherboard), power, state (powersupply, HDD, Fans, memory slots) sensors support. Tested with HP ProLiant ML350p Gen8-Gen9
is this ok to test pull on? |
Auto-Deploy finished, Test PR at http://7053.ci.librenms.org or https://7053.ci.librenms.org |
@kkrumm1 it is now. |
Auto-Deploy finished, Test PR at http://7053.ci.librenms.org or https://7053.ci.librenms.org |
The inspection completed: No new issues |
Receiving this error when I git pull "error: mibs/CPQHLTH-MIB: No such file or directory" |
That's been moved, have you checked my branch out? |
i don't think so all did was ./scripts/github-apply 7053 |
That might not work well with files being moved around :/ |
Oh, but you don't need to git pull when you've run the apply script, the script is all you need |
Ohhhhhh okay
…On Jul 18, 2017 5:08 PM, "Neil Lathwood" ***@***.***> wrote:
Oh, but you don't need to git pull when you've run the apply script, the
script is all you need
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#7053 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/Aaqge69TxKaVBxfNpIl5AuOmZ5gX2btVks5sPSzrgaJpZM4Ob2K3>
.
--
CONFIDENTIALITY NOTICE: This email & attached documents may contain
confidential information. All information is intended only for the use of
the named recipient. If you are not the named recipient, you are not
authorized to read, disclose, copy, distribute or take any action in
reliance on the information and any action other than immediate delivery to
the named recipient is strictly prohibited. If you have received this email
in error, do not read the information and please immediately notify sender
by telephone to arrange for a return of the original documents. If you are
the named recipient you are not authorized to reveal any of this
information to any other unauthorized person. If you did not receive all
pages listed or if pages are not legible, please immediately notify sender
by phone.
|
@laf Here is what I'm seeing https://pastebin.com/qsWkTzQF |
That doesn't seem to show the end. Sometimes the apply script isn't the best to cover things where files are moved around so if you want to test properly I suggest checking out my branch. |
OK - it just cut off this last line "error: mibs/hp/CPQHLTH-MIB: No such
file or directory"
…On Tue, Jul 18, 2017 at 6:26 PM, Neil Lathwood ***@***.***> wrote:
That doesn't seem to show the end. Sometimes the apply script isn't the
best to cover things where files are moved around so if you want to test
properly I suggest checking out my branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#7053 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AaqgexUqE5iankkKIxIJq3Z0d8WuPgeYks5sPT8ggaJpZM4Ob2K3>
.
--
Kevin Krumm
Jr Network/Systems Admin
New Braunfels ISD Technology Services
(830) 643-5759
"There are 10 types of people in the world: Those who understand binary,
and those who don't."
--
CONFIDENTIALITY NOTICE: This email & attached documents may contain
confidential information. All information is intended only for the use of
the named recipient. If you are not the named recipient, you are not
authorized to read, disclose, copy, distribute or take any action in
reliance on the information and any action other than immediate delivery to
the named recipient is strictly prohibited. If you have received this email
in error, do not read the information and please immediately notify sender
by telephone to arrange for a return of the original documents. If you are
the named recipient you are not authorized to reveal any of this
information to any other unauthorized person. If you did not receive all
pages listed or if pages are not legible, please immediately notify sender
by phone.
|
Do you want that I migrate the code to the new YAML format? [EDIT] PR mix, it's already done! Wonderful. [/EDIT] |
This thread has been automatically locked since there has not been any recent activity after it was closed. |
DO NOT DELETE THIS 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
Replaces: #6996