Fix Brocade VDX detection #3888

Merged
merged 2 commits into from Jul 28, 2016

Projects

None yet

4 participants

@xbeaudouin
Contributor

Seems that brocade has changed some strings again. This trivial patch fix this on VDX6740, BR-VDX6740, Network Operating System Software Version 6.0.2a2

xavier.beaudouin Fix Brocade VDX detection on VDX6740, BR-VDX6740, Network Operating S…
…ystem Software Version 6.0.2a2
ac6bca4
@Rosiak Rosiak commented on an outdated diff Jul 27, 2016
includes/discovery/os/nos.inc.php
@@ -4,4 +4,7 @@
if (strstr($sysDescr, "Brocade VDX")) {
@Rosiak
Rosiak Jul 27, 2016 Contributor

You could just do
if (strstr($sysDescr, "Brocade VDX") II strstr($sysDescr, "BR-VDX")) {

xavier.beaudouin Better detection and adding VDX with NOS 3.0.1a detection too (seems …
…that Brocade has decided to be like Cisco on their OS : more broken every time).
b19fae2
@laf laf merged commit 6f55e22 into librenms:master Jul 28, 2016

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