Fix Dell PowerConnect Discovery #3998

Merged
merged 2 commits into from Aug 8, 2016

Projects

None yet

4 participants

@Rosiak
Contributor
Rosiak commented Aug 7, 2016

Please note

Please read this information carefully.

[X] Have you signed the Contributors agreement

[X] Have you followed our code guidelines?
#3995

This seems to be some leftover from the fork.

@Rosiak Rosiak Fix Dell PowerConnect Discovery
d65364c
@murrant
Contributor
murrant commented Aug 8, 2016

There is a bit in the includes/polling/osradlan.inc.php file too, can you remove that?

@Rosiak Rosiak Remove more code
057d78a
@murrant murrant merged commit 57661cf into librenms:master Aug 8, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mzac
mzac commented Aug 9, 2016

I did an update tonight and now it is reporting another manufacturer:

2016-08-08 19:50:03 Icon -> allied  
2016-08-08 19:50:03 Hardware -> Neyland 24T 
2016-08-08 18:40:03 Icon -> dell    
2016-08-08 18:35:41 Device Icon changed dell => radlan
Version info:
Commit SHA: b93a060458d0d65447973ecacc6f326fbdbcb936
DB Schema: 126
PHP: 5.6.24-0+deb8u1
MySQL: 5.5.50-0+deb8u1
RRDTool: 1.4.8
SNMP: NET-SNMP version: 5.7.2.1
@Rosiak Rosiak deleted the Rosiak:issue-3995 branch Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment