Add support for some UPS that use same card as multimatic #2046

Merged
merged 4 commits into from Oct 7, 2015

Projects

None yet

4 participants

@xbeaudouin
Contributor

There is some UPS that have same management card as a multimatic that use UPS-MIB.
This pull request add "simple" way to support them.

xavier.beaud... added some commits Oct 2, 2015
xavier.beaudouin First portage of S2S/Mulitmatic support for S2S SYRIUS MS 10000 22f5266
xavier.beaudouin Merge branch 'master' of https://github.com/librenms/librenms
a4694c7
@laf
Member
laf commented Oct 3, 2015

Hi @xbeaudouin

We will need you to sign the contributors agreement, an example of a commit is here: paszczus@75119b1

@xbeaudouin
Contributor

Do I have to make another pull request for that ?

@f0o
Member
f0o commented Oct 6, 2015

@xbeaudouin preferably yes but you can also attach it to this one as it's quite small.
Can you rebase your branch against upstream/master before pushing again? It's to get rid of the 'merge' commit you got there. (you might need to force-push after rebasing)

@f0o f0o self-assigned this Oct 6, 2015
@f0o f0o added the Discovery label Oct 6, 2015
xavier.beaud... added some commits Oct 6, 2015
xavier.beaudouin Merge branch 'master' of https://github.com/librenms/librenms b7e0a9f
xavier.beaudouin I agree to the conditions of the Contributor Agreement contained in d…
…oc/General/Contributing.md
10c5972
@xbeaudouin
Contributor

Agreement done.

@laf laf merged commit 738db49 into librenms:master Oct 7, 2015

2 checks passed

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