Fix vs test issue: failed to remove vlan due to referenced by vlan in…

…terface (#2504) *Fix: remove vlan interface before removing the vlan itself
sonic-net · Nov 29, 2022 · 434e80c · 434e80c
1 parent 11bef87
commit 434e80c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/tests/dvslib/dvs_vlan.py b/tests/dvslib/dvs_vlan.py
@@ -46,6 +46,10 @@ def remove_vlan_member(self, vlanID, interface):
         member = "Vlan{}|{}".format(vlanID, interface)
         self.config_db.delete_entry("VLAN_MEMBER", member)
 
+    def remove_vlan_interface(self, vlanID):
+        vlan = "Vlan{}".format(vlanID)
+        self.config_db.delete_entry("VLAN_INTERFACE", vlan)
+
     def check_app_db_vlan_fields(self, fvs, admin_status="up", mtu="9100"):
         assert fvs.get("admin_status") == admin_status
         assert fvs.get("mtu") == mtu

diff --git a/tests/test_vlan.py b/tests/test_vlan.py
@@ -459,6 +459,7 @@ def arp_accept_disabled():
 
         wait_for_result(arp_accept_disabled, PollingConfig(), "IPv4 arp_accept not disabled")
 
+        self.dvs_vlan.remove_vlan_interface(vlan)
         self.dvs_vlan.remove_vlan(vlan)
 
     def test_VlanProxyArp(self, dvs): 
@@ -486,6 +487,7 @@ def proxy_arp_disabled():
 
         wait_for_result(proxy_arp_disabled, PollingConfig(), 'IPv4 proxy_arp or proxy_arp_pvlan not disabled')
 
+        self.dvs_vlan.remove_vlan_interface(vlan)
         self.dvs_vlan.remove_vlan(vlan)
 
 # Add Dummy always-pass test at end as workaroud