Skip to content

Commit e04bb43

Browse files
authored
Fix vs test issue: failed to remove vlan due to referenced by vlan interface (sonic-net#2504)
*Fix: remove vlan interface before removing the vlan itself
1 parent 52c561f commit e04bb43

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

tests/dvslib/dvs_vlan.py

+4
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,10 @@ def remove_vlan_member(self, vlanID, interface):
4646
member = "Vlan{}|{}".format(vlanID, interface)
4747
self.config_db.delete_entry("VLAN_MEMBER", member)
4848

49+
def remove_vlan_interface(self, vlanID):
50+
vlan = "Vlan{}".format(vlanID)
51+
self.config_db.delete_entry("VLAN_INTERFACE", vlan)
52+
4953
def check_app_db_vlan_fields(self, fvs, admin_status="up", mtu="9100"):
5054
assert fvs.get("admin_status") == admin_status
5155
assert fvs.get("mtu") == mtu

tests/test_vlan.py

+2
Original file line numberDiff line numberDiff line change
@@ -459,6 +459,7 @@ def arp_accept_disabled():
459459

460460
wait_for_result(arp_accept_disabled, PollingConfig(), "IPv4 arp_accept not disabled")
461461

462+
self.dvs_vlan.remove_vlan_interface(vlan)
462463
self.dvs_vlan.remove_vlan(vlan)
463464
self.dvs_vlan.get_and_verify_vlan_ids(0)
464465

@@ -487,6 +488,7 @@ def proxy_arp_disabled():
487488

488489
wait_for_result(proxy_arp_disabled, PollingConfig(), 'IPv4 proxy_arp or proxy_arp_pvlan not disabled')
489490

491+
self.dvs_vlan.remove_vlan_interface(vlan)
490492
self.dvs_vlan.remove_vlan(vlan)
491493
self.dvs_vlan.get_and_verify_vlan_ids(0)
492494

0 commit comments

Comments
 (0)