Skip to content

Commit

Permalink
fix: Link aggregation member ports don’t have VLANs, don’t try to delete
Browse files Browse the repository at this point in the history
  • Loading branch information
ypid committed Nov 1, 2021
1 parent f1f5634 commit b676d4a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -141,8 +141,8 @@ def get_interfaces_vlan_membership(self):
def get_interface_vlan_membership_change_actions(self, interface, port_vlans, desired_port_vlans):
change_actions = []
vlan_vids = [int(vlan[0]) for vlan in self.getVlans() if len(vlan) == 3]
if 'untagged' in desired_port_vlans:
if desired_port_vlans['untagged'] is None and 'untagged' in port_vlans:
if 'untagged' in desired_port_vlans and 'untagged' in port_vlans:
if desired_port_vlans['untagged'] is None:
# The switch does not support that:
# > If you wish to exclude a port from the current VLAN
# > membership you must first make it a tagged/untagged member in
Expand Down

0 comments on commit b676d4a

Please sign in to comment.