start of adding basic support for the servertech sentry4 cdus #3659

Merged
merged 4 commits into from Jun 15, 2016

Projects

None yet

4 participants

@justmedude
Contributor

(let's try this again)

With the servertech pdus, the OID mappings between the sentry3 and the sentry4 line is significantly different. Enough so that you really need to handle it differently. This diff hopefully provide some of the initial forking in code path for sentry3 vs. 4. My primary needs are to get temp data which is why only that section is being handled.

Feel free to completely reject the pull request. I did want to at least contribute some of the bare bits to get others started.

justmedude added some commits Jun 15, 2016
@justmedude justmedude start of adding basic support for the servertech sentry4 cdus 5e58604
@justmedude justmedude start of adding basic support for the servertech sentry4 cdus
    I agree to the conditions of the Contributor Agreement
    contained in doc/General/Contributing.md.
5b9fb7d
@justmedude justmedude Merge branch 'master' of github.com:justmedude/librenms
7d61f21
@f0o f0o added the New-Device label Jun 15, 2016
@f0o
Member
f0o commented Jun 15, 2016

Looks OK to me 👍

@laf
Member
laf commented Jun 15, 2016

Just needs you to sign the contributors agreement as per the doc, the commit message has to contain the specific text.

You can just put a space next to your name and do it in that commit to this pr.

Aside from that, +1.

@justmedude justmedude I agree to the conditions of the Contributor Agreement
    contained in doc/General/Contributing.md.
d247af2
@laf laf merged commit 66f8a7e into librenms:master Jun 15, 2016

1 of 2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment