snmp-info icon indicating copy to clipboard operation
snmp-info copied to clipboard

CiscoSB needs distinct Layer2 and Layer3 packages

Open nakaori opened this issue 7 years ago • 7 comments

Expected Behavior

pulling arp table from layer3 ciscosb switch

Current Behavior

netdisco is using layer 2, CiscoSB -> [64660] 2018-11-15 11:22:35 debug [192.168.1.254:161] try_connect with ver: 2, class: SNMP::Info::Layer2::CiscoSB, comm:

Old installation of netdisco is using layer 3, Cisco -> [8362] 2018-11-15 11:27:37 debug [192.168.1.254] try_connect with ver: 2, class: SNMP::Info::Layer3::Cisco, comm:

Possible Solution

Steps to Reproduce (for bugs)

Install new version of Netdisco as per tutorial

Context

I want the arp table of my layer3 environment to be pulled.

Your Environment

  • Netdisco version used: 2.39.33
  • SNMP::Info version used: 3.61

nakaori avatar Nov 15 '18 11:11 nakaori

Hi, please can you give the Netdisco and SNMP::Info version of your old installation where it's working correctly?

Thanks

ollyg avatar Jan 02 '19 19:01 ollyg

Hi @ollyg

SNMP::Info | 3.39

nakaori avatar Jan 22 '19 08:01 nakaori

the only things i think that might have an impact are commits

netdisco/snmp-info@02a52a4e09f7d24a7b3e6bdb7b010beeb31b5627 netdisco/snmp-info@e75aba14b4220fb78fb660029213d772bacdff68

inphobia avatar Jan 25 '19 21:01 inphobia

Thanks @inphobia, looks like CiscoSB is a Layer2 implementation but is used even if the device has Layer3 capabilities. So we need a Layer3 version of CiscoSB (or use another such as CiscoSwitch).

I'd like @JeroenvIS or @jeneric to advise - I really don't understand the SNMP::Info layers inheritance model sufficiently.

ollyg avatar Jan 26 '19 22:01 ollyg

i'll tag @postilion too, i think he did most of the work for ciscosb support.

one question i do have: https://github.com/netdisco/snmp-info/blob/master/lib/SNMP/Info.pm#L1833

why .?1: $soid =~ /^\.?1

shouldn't $soid =~ /^\.1 be ok? all other oid based checks expect .1 to be there.

inphobia avatar Jan 27 '19 01:01 inphobia

good question, I've no idea :-D. but it can't hurt, I guess.

On Sun, 27 Jan 2019 at 01:13, nick n. [email protected] wrote:

i'll tag @postilion https://github.com/postilion too, i think he did most of the work for ciscosb support.

one question i do have: https://github.com/netdisco/snmp-info/blob/master/lib/SNMP/Info.pm#L1833

why ?1: $soid =~ /^.?1

shouldn't $soid =~ /^.1 be ok? all other oid based checks expect .1 to be there.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/netdisco/snmp-info/issues/300#issuecomment-457880195, or mute the thread https://github.com/notifications/unsubscribe-auth/AABz1dz-LNp1ibUaQ8PnHzKOge_-TASVks5vHP0zgaJpZM4aUcjd .

ollyg avatar Jan 27 '19 11:01 ollyg

@nakaori would is first be possible to upgrade to the latest netdisco & snmp::info?

after that we could use the output of: netdisco-do show -d <ip> -e ::layers netdisco-do show -d <ip> -e layers

the ::layers version was only added recently, thats why i ask to upgrade.

after that i would like to see if the fix for dlink mentioned in https://github.com/netdisco/netdisco-mibs/issues/63#issuecomment-459320044 might fix the issue.

perhaps rebuilding ciscosb to a layer3 module is also an option, but that will take time & testing.

inphobia avatar Jan 31 '19 14:01 inphobia