Eaton UPS - Current Discovery #3066

Merged
merged 1 commit into from Feb 24, 2016

Projects

None yet

4 participants

@GeorgesDick
Contributor

I added this file in my local LibreNMS installation to get current values from my Eaton 5PX UPS. Another file is also needed :
librenms/includes/discovery/os/eatonups.inc.php

@GeorgesDick GeorgesDick Added eatonups.inc.php
I added this file in my local LibreNMS installation to get current values from my Eaton 5PX UPS. Another file is also needed :
librenms/includes/discovery/os/eatonups.inc.php
a8c4fe8
@laf
Member
laf commented Feb 23, 2016

Thanks. Can you sign the contributors agreement please as per: http://docs.librenms.org/General/Contributing/

@GeorgesDick
Contributor

Thank you @laf I added my name in AUTHORS.md, but can't find how to agree to the conditions of the Contributor Agreement in the commit log: where is the commit log and how to edit it? (Sorry, I'm used to Subversion, I'm only a beginner using Git)

@Rosiak Rosiak changed the title from Added eatonups.inc.php to Eaton UPS - Current Discovery Feb 23, 2016
@paulgear
Member

@GeorgesDick The commit log is the message you use in committing your change. So in #3073, instead of saying "Update AUTHORS.md", you should include the text mentioned in the contributor agreement.

@GeorgesDick
Contributor

OK, done, hope I did it right.

@laf laf merged commit 92df310 into librenms:master Feb 24, 2016

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer 2 new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment