openstack/kayobe
Code Issues Proposed changes

Merge "Filter out switch port descriptions on Ruckus switches"

Browse Source
This commit is contained in:
Zuul 2020-04-09 15:33:41 +00:00 committed by Gerrit Code Review
commit 9d5a671f3c
2 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
ansible/group_vars/all/inspector
View File

@ -194,24 +194,25 @@ inspector_rule_lldp_switch_port_desc_to_name:
op: "is-empty" op: "is-empty"
invert: True invert: True
# Filter out switch port descriptions using the switch's interface names. # Filter out switch port descriptions using the switch's interface names.
# On some switches (including Dell Network OS 9.10(0.1)), the port # On some switches (including Dell Network OS 9.10(0.1) and some Ruckus
# description TLV is sent but contains the interface name rather than the # switches), the port description TLV is sent but contains the interface
# interface's description. # name rather than the interface's description. Dell switches use a space
# character between port type and port number, while Ruckus switches don't.
- field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description" - field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description"
op: "matches" op: "matches"
value: "^GigabitEthernet ([0-9/]*)$" value: "^GigabitEthernet ?([0-9/]*)$"
invert: True invert: True
- field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description" - field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description"
op: "matches" op: "matches"
value: "^TenGigabitEthernet ([0-9/]*)$" value: "^TenGigabitEthernet ?([0-9/]*)$"
invert: True invert: True
- field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description" - field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description"
op: "matches" op: "matches"
value: "^fortyGigE ([0-9/]*)$" value: "^fortyGigE ?([0-9/]*)$"
invert: True invert: True
- field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description" - field: "data://all_interfaces.{{inspector_rule_var_lldp_switch_port_interface}}.lldp_processed.switch_port_description"
op: "matches" op: "matches"
value: "^Port-channel ([0-9/]*)$" value: "^Port-channel ?([0-9/]*)$"
invert: True invert: True
actions: actions:
- action: "set-attribute" - action: "set-attribute"

6
releasenotes/notes/filter-out-ruckus-lldp-port-description-ad2b0774609aedbf.yaml Normal file
View File

@ -0,0 +1,6 @@
---
issues:
- |
Fixes an issue where the default inspector rule setting node names from
LLDP switch port descriptions would fail to filter out interface names on
Ruckus switches.