Add sysObjectId for Mellanox i5035 infiniband switch #5887

Merged
merged 6 commits into from Feb 15, 2017

Conversation

Projects
None yet
5 participants
@mobilemaclin
Contributor

mobilemaclin commented Feb 14, 2017

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

  • Have you signed the Contributors agreement - please do NOT submit a pull request unless you have (signing the agreement in the same pull request is fine). Your commit message for signing the agreement must appear as per the docs.
  • Have you followed our code guidelines?
@mobilemaclin

This comment has been minimized.

Show comment
Hide comment
@mobilemaclin

mobilemaclin Feb 14, 2017

Owner

I had to add this sysObjectId for my mellanox i5035 infiniband switch so it would not show as a linux server

Owner

mobilemaclin commented on a181ce6 Feb 14, 2017

I had to add this sysObjectId for my mellanox i5035 infiniband switch so it would not show as a linux server

@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot Feb 14, 2017

Thank you for submitting a PR @mobilemaclin! We have found the following @laf based on the history of these files to review this PR.

Thank you for submitting a PR @mobilemaclin! We have found the following @laf based on the history of these files to review this PR.

@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 14, 2017

Member

Can you provide the output of sysDescr so we can add a unit test please @mobilemaclin

Member

laf commented Feb 14, 2017

Can you provide the output of sysDescr so we can add a unit test please @mobilemaclin

@mobilemaclin

This comment has been minimized.

Show comment
Hide comment
@mobilemaclin

mobilemaclin Feb 14, 2017

Contributor

Linux Mellanox 2.6.27-MELLANOXuni-m460ex EFM_PPC_M460EX EFM_1.1.2910 #1 2012-07-11 13:55:43 ppc

Let me know if you mean something else, this is all little new to me

Contributor

mobilemaclin commented Feb 14, 2017

Linux Mellanox 2.6.27-MELLANOXuni-m460ex EFM_PPC_M460EX EFM_1.1.2910 #1 2012-07-11 13:55:43 ppc

Let me know if you mean something else, this is all little new to me

@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

scrutinizer-notifier Feb 14, 2017

The inspection completed: No new issues

The inspection completed: No new issues

@laf laf merged commit 49b5844 into librenms:master Feb 15, 2017

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