diff options
-rw-r--r-- | python/vyos/ifconfig/bridge.py | 9 | ||||
-rw-r--r-- | python/vyos/ifconfig/interface.py | 9 |
2 files changed, 6 insertions, 12 deletions
diff --git a/python/vyos/ifconfig/bridge.py b/python/vyos/ifconfig/bridge.py index 85b5335de..a565d37a6 100644 --- a/python/vyos/ifconfig/bridge.py +++ b/python/vyos/ifconfig/bridge.py @@ -342,10 +342,6 @@ class BridgeIf(Interface): vlan_id = interface_config['native_vlan'] add_vlan.append(vlan_id) native_vlan_id = vlan_id - else: - # VLAN 1 is the default VLAN for all unlabeled packets - add_vlan.append(1) - native_vlan_id = 1 if 'allowed_vlan' in interface_config: for vlan in interface_config['allowed_vlan']: @@ -367,8 +363,9 @@ class BridgeIf(Interface): cmd = f'bridge vlan add dev {interface} vid {vlan} master' self._cmd(cmd) # Setting native VLAN to system - cmd = f'bridge vlan add dev {interface} vid {native_vlan_id} pvid untagged master' - self._cmd(cmd) + if native_vlan_id: + cmd = f'bridge vlan add dev {interface} vid {native_vlan_id} pvid untagged master' + self._cmd(cmd) # Enable/Disable of an interface must always be done at the end of the # derived class to make use of the ref-counting set_admin_state() diff --git a/python/vyos/ifconfig/interface.py b/python/vyos/ifconfig/interface.py index f7578fe2d..3b92ce463 100644 --- a/python/vyos/ifconfig/interface.py +++ b/python/vyos/ifconfig/interface.py @@ -912,10 +912,6 @@ class Interface(Control): vlan_id = bridge_config['native_vlan'] add_vlan.append(vlan_id) native_vlan_id = vlan_id - else: - # VLAN 1 is the default VLAN for all unlabeled packets - add_vlan.append(1) - native_vlan_id = 1 if 'allowed_vlan' in bridge_config: for vlan in bridge_config['allowed_vlan']: @@ -937,8 +933,9 @@ class Interface(Control): cmd = f'bridge vlan add dev {ifname} vid {vlan} master' self._cmd(cmd) # Setting native VLAN to system - cmd = f'bridge vlan add dev {ifname} vid {native_vlan_id} pvid untagged master' - self._cmd(cmd) + if native_vlan_id: + cmd = f'bridge vlan add dev {ifname} vid {native_vlan_id} pvid untagged master' + self._cmd(cmd) def set_dhcp(self, enable): """ |