Skip to content

Commit f3da40c

Browse files
committed
B #6985: Prevent race condition errors with keep_empty_bridge
1 parent 5d6878d commit f3da40c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/vnm_mad/remotes/ovswitch/OpenvSwitch.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,10 @@ class OpenvSwitchVLAN < VNMMAD::VNMDriver
3939
]
4040

4141
def initialize(vm, xpath_filter = nil, deploy_id = nil)
42-
@locking = false
43-
4442
xpath_filter ||= XPATH_FILTER
4543
super(vm, xpath_filter, deploy_id)
44+
45+
@locking = false
4646
end
4747

4848
def activate
@@ -550,7 +550,7 @@ def create_bridge
550550

551551
# Delete OvS bridge
552552
def delete_bridge
553-
LocalCommand.run_sh("#{command(:ovs_vsctl)} del-br #{@nic[:bridge]}")
553+
LocalCommand.run_sh("#{command(:ovs_vsctl)} --if-exists del-br #{@nic[:bridge]}")
554554

555555
@bridges.delete(@nic[:bridge])
556556
end
@@ -576,7 +576,7 @@ def del_bridge_port(port)
576576
LocalCommand.run_sh("#{command(:ovs_vsctl)} --if-exists del-port " \
577577
"#{@nic[:bridge]} #{port}")
578578

579-
@bridges[@nic[:bridge]].delete(port)
579+
@bridges[@nic[:bridge]]&.delete(port)
580580
end
581581

582582
# Calls ovs-vsctl set bridge to set options stored in ovs_bridge_conf

0 commit comments

Comments
 (0)