New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avaya switch 4950GTS-PWR-PLUS detection wrong. #239

Closed
zoeloe opened this Issue Dec 20, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@zoeloe

zoeloe commented Dec 20, 2017

Expected Behavior

Interfaces should be numbered correctly.

Current Behavior

From a stack with 4 units of 4950GTS-PWR-PLUS, the detected ports in netdisco are numbered:
1.[1-50]
3.[1-50]
5.[1-50]
7.[1-50]

PoE capable ports from units 2 and 4 are not detected ("0" ports detected)

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Switch/Stack: SW version: boss / 7.4.0.009
SNMP::Info version 3.39
App-Netdisco-2.037003

@zoeloe

This comment has been minimized.

Show comment
Hide comment
@zoeloe

zoeloe Dec 20, 2017

FYI: seems to be related to #238 and #222.
librenms detects port and units correctly.

zoeloe commented Dec 20, 2017

FYI: seems to be related to #238 and #222.
librenms detects port and units correctly.

@zoeloe

This comment has been minimized.

Show comment
Hide comment
@zoeloe

zoeloe Jan 16, 2018

Suggested intermediate fix:
It seems that a 4950 switch has a index_factor of 128.
It looks that modifying SNMP::Info::Layer2::Baystack.pm

the diff is e.g.:

----------------
232c232
<         if ( ( $model =~ /(5[56]\d\d)|VSP|(4950)/ )
---
>         if ( ( $model =~ /(5[56]\d\d)|VSP/ )
----------------

does the trick.
It would be nice if a generic catch-all fix would be available. A similar regex would catch
other devices with an index_factor of 128.

zoeloe commented Jan 16, 2018

Suggested intermediate fix:
It seems that a 4950 switch has a index_factor of 128.
It looks that modifying SNMP::Info::Layer2::Baystack.pm

the diff is e.g.:

----------------
232c232
<         if ( ( $model =~ /(5[56]\d\d)|VSP|(4950)/ )
---
>         if ( ( $model =~ /(5[56]\d\d)|VSP/ )
----------------

does the trick.
It would be nice if a generic catch-all fix would be available. A similar regex would catch
other devices with an index_factor of 128.

@ollyg

This comment has been minimized.

Show comment
Hide comment
@ollyg

ollyg Feb 12, 2018

Member

fixed in 4afbd37, many thanks!!

Member

ollyg commented Feb 12, 2018

fixed in 4afbd37, many thanks!!

@ollyg ollyg closed this Feb 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment