netdisco / snmp-info

Other
39 stars 32 forks source link

Arista vlan change on port: VLAN set not supported. #429

Open jacobghattas opened 3 years ago

jacobghattas commented 3 years ago

Greeting, I am having a bit of troubles getting the VLAN changes done with switches like Ruckus (ICS7K EX-Foundry) and Arista.

Getting this output when changing trying to do the change via netdisco-do.

I have tickets opened both with Ruckus and Arista tac about it.

would appreciate any help.

[netdisco@netlogdbgsrv ~]$ ~/bin/netdisco-do vlan -d 10.250.99.4 -p Ethernet11 -e 197 -DI [7044] 2021-07-09 05:49:55 info App::Netdisco version 2.047005 loaded. [7044] 2021-07-09 05:49:55 info vlan: [10.250.99.4] started at Fri Jul 9 08:49:55 2021 [7044] 2021-07-09 05:49:55 debug vlan: running with timeout 600s [7044] 2021-07-09 05:49:55 debug => running workers for phase: check [7044] 2021-07-09 05:49:55 debug -> run worker check/base/0 [7044] 2021-07-09 05:49:55 debug Vlan is able to run. [7044] 2021-07-09 05:49:55 debug => running workers for phase: early [7044] 2021-07-09 05:49:55 debug -> run worker early/core/100 [7044] 2021-07-09 05:49:55 debug snmp writer cache warm: [10.250.99.4] [7044] 2021-07-09 05:49:55 debug [10.250.99.4:161] try_connect with ver: 2, class: SNMP::Info::Layer3::Arista, comm: SNMP::Info::_global load_location : SNMPv2-MIB::sysLocation.0 : .1.3.6.1.2.1.1.6.0 SNMP::Info::_set set_location.0 (.1.3.6.1.2.1.1.6.0) = SNMP::Info::_global load_location : SNMPv2-MIB::sysLocation.0 : .1.3.6.1.2.1.1.6.0 SNMP::Info::_global layers : SNMPv2-MIB::sysServices.0 : .1.3.6.1.2.1.1.7.0 SNMP::Info::_global description : SNMPv2-MIB::sysDescr.0 : .1.3.6.1.2.1.1.1.0 SNMP::Info::_global id : SNMPv2-MIB::sysObjectID.0 : .1.3.6.1.2.1.1.2.0 SNMP::Info 3.71 SNMP::Info::device_type() layers:00001110 id:30065 sysDescr:"Arista Networks EOS version 4.23.4M running on an Arista Networks DCS-7050SX3-48YC8" SNMP::Info::_load_attr i_index : IF-MIB::ifIndex : .1.3.6.1.2.1.2.2.1.1 SNMP::Info::_load_attr i_description : IF-MIB::ifDescr : .1.3.6.1.2.1.2.2.1.2 [7044] 2021-07-09 05:49:56 debug Vlan set can continue. [7044] 2021-07-09 05:49:56 debug => running workers for phase: main [7044] 2021-07-09 05:49:56 debug -> run worker main/core/100 VLAN set not supported. at /home/netdisco/perl5/lib/perl5/App/Netdisco/Worker/Plugin/Vlan/Core.pm line 42. VLAN set not supported. at /home/netdisco/perl5/lib/perl5/App/Netdisco/Worker/Plugin/Vlan/Core.pm line 42. [7044] 2021-07-09 05:49:56 debug Failed to set [Ethernet11] vlan to [197] on $device: VLAN set not supported. [7044] 2021-07-09 05:49:56 info vlan: finished at Fri Jul 9 08:49:56 2021 [7044] 2021-07-09 05:49:56 info vlan: status info: Vlan set can continue.

JeroenvIS commented 3 years ago

Since this ticket has some debug output for the Arista case, I'll respond to that part. Perhaps it's better to create a separate ticket for the Ruckus, also with debug output to support it.

As for the Arista: looking at both the debug output and the code, currently VLAN changes on Arista aren't supported by SNMP::Info (and SNMP::Info is the SNMP interface layer that Netdisco uses for these interactions).

Now I'm not sure why this wasn't implemented. If Arista switches don't support VLAN changes via SNMP in the first place, then that's a good reason, although it would be cleaner to throw a more specific error. If the switches do support it, then we only need to add a few bits of code.

Do you happen to know whether VLAN settings are SNMP writable on Arista?

jacobghattas commented 3 years ago

Hello Jeroen, Thanks for the response. I am checking it with arista's tac. No evident response yet.

Hope will get an official one soon and update.

10xs. Jacob

בתאריך יום ד׳, 14 ביולי 2021, 10:04, מאת Jeroen van Ingen Schenau ‏< @.***>:

Since this ticket has some debug output for the Arista case, I'll respond to that part. Perhaps it's better to create a separate ticket for the Ruckus, also with debug output to support it.

As for the Arista: looking at both the debug output and the code, currently VLAN changes on Arista aren't supported by SNMP::Info (and SNMP::Info is the SNMP interface layer that Netdisco uses for these interactions).

Now I'm not sure why this wasn't implemented. If Arista switches don't support VLAN changes via SNMP in the first place, then that's a good reason, although it would be cleaner to throw a more specific error. If the switches do support it, then we only need to add a few bits of code.

Do you happen to know whether VLAN settings are SNMP writable on Arista?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/netdisco/snmp-info/issues/429#issuecomment-879646957, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMUYXU7IQKH6NZQQ4NPUE53TXUZGNANCNFSM5AJXGJMQ .

JeroenvIS commented 3 years ago

Looks like I have 1 Arista in our network, first test looks promising

JeroenvIS commented 3 years ago

OK, got it working on Arista with a simple patch. I'll change the title for this issue to only cover Arista; please add a new ticket for Ruckus/Foundry with relevant debug output.

JeroenvIS commented 3 years ago

Patch pushed to branch in commit be1b8fd

jacobghattas commented 3 years ago

Great news 👏. Really appreciate it.

בתאריך יום ד׳, 14 ביולי 2021, 11:51, מאת Jeroen van Ingen Schenau ‏< @.***>:

Patch pushed to branch in commit be1b8fd https://github.com/netdisco/snmp-info/commit/be1b8fda66974fa1118cfdcafb3291af4579441f

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/netdisco/snmp-info/issues/429#issuecomment-879716492, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMUYXU2H3VZUF5QY66RCR33TXVFXPANCNFSM5AJXGJMQ .

jacobghattas commented 3 years ago

Sure thing, will do shortly.

Should I do it on netdisco or netdisco-snmp?

בתאריך יום ד׳, 14 ביולי 2021, 11:35, מאת Jeroen van Ingen Schenau ‏< @.***>:

OK, got it working on Arista with a simple patch. I'll change the title for this issue to only cover Arista; please add a new ticket for Ruckus/Foundry with relevant debug output.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/netdisco/snmp-info/issues/429#issuecomment-879706442, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMUYXUZO5VGB3L7KJKDMTJDTXVD5NANCNFSM5AJXGJMQ .

jacobghattas commented 3 years ago

Snmp-info I mean?.

בתאריך יום ה׳, 15 ביולי 2021, 21:19, מאת Jacob Ghattas ‏< @.***>:

Sure thing, will do shortly.

Should I do it on netdisco or netdisco-snmp?

בתאריך יום ד׳, 14 ביולי 2021, 11:35, מאת Jeroen van Ingen Schenau ‏< @.***>:

OK, got it working on Arista with a simple patch. I'll change the title for this issue to only cover Arista; please add a new ticket for Ruckus/Foundry with relevant debug output.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/netdisco/snmp-info/issues/429#issuecomment-879706442, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMUYXUZO5VGB3L7KJKDMTJDTXVD5NANCNFSM5AJXGJMQ .

JeroenvIS commented 3 years ago

Snmp-info I mean?

Yes, an issue against SNMP::Info please, with similar "netdisco-do vlan" etc debug output.

jacobghattas commented 3 years ago

Jeroen, Issue 433 opened for ruckus. Thank u.