OpenNebula#3058 del_bridge_port returns only boolean

dann1 · Mar 22, 2019 · dab395c · dab395c
1 parent 78477c2
commit dab395c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/vmm_mad/remotes/lib/lxd/container.rb b/src/vmm_mad/remotes/lib/lxd/container.rb
@@ -392,7 +392,7 @@ def vnc(signal)
 
     # Deletes the switch port. Unlike libvirt, LXD doesn't handle this.
     def del_bridge_port(nic)
-        return 'not ovswitch' unless nic['VN_MAD'] == 'ovswitch'
+        return true unless nic['VN_MAD'] == 'ovswitch'
 
         cmd = 'sudo ovs-vsctl --if-exists del-port '\
         "#{nic['BRIDGE']} #{nic['TARGET']}"
@@ -402,7 +402,7 @@ def del_bridge_port(nic)
         return true if rc.zero?
 
         OpenNebula.log_error "#{__method__}: #{e}"
-        nil
+        false
     end
 
     # Waits or no for response depending on wait value