From 88331de3fb9375d316acba7b160ddf2a9beab5b2 Mon Sep 17 00:00:00 2001 From: Nikolas Hermanns Date: Wed, 3 May 2017 17:49:53 +0200 Subject: Clean always and change order Change-Id: Ic9e0d374a4fe5278888b28fe2694c546840d55b4 Signed-off-by: Nikolas Hermanns --- sdnvpn/lib/utils.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) (limited to 'sdnvpn/lib/utils.py') diff --git a/sdnvpn/lib/utils.py b/sdnvpn/lib/utils.py index 5eeb667..59f641b 100644 --- a/sdnvpn/lib/utils.py +++ b/sdnvpn/lib/utils.py @@ -509,6 +509,14 @@ def attach_instance_to_ext_br(instance, compute_node): def detach_instance_from_ext_br(instance, compute_node): libvirt_instance_name = getattr(instance, "OS-EXT-SRV-ATTR:instance_name") + mac = compute_node.run_cmd("for vm in $(sudo virsh list | " + "grep running | awk '{print $2}'); " + "do echo -n ; sudo virsh dumpxml $vm| " + "grep -oP '52:54:[\da-f:]+' ;done") + compute_node.run_cmd("sudo virsh detach-interface --domain %s" + " --type bridge --mac %s" + % (libvirt_instance_name, mac)) + installer_type = str(os.environ['INSTALLER_TYPE'].lower()) if installer_type == "fuel": bridge = "br-ex" @@ -528,11 +536,3 @@ def detach_instance_from_ext_br(instance, compute_node): sudo brctl delbr {bridge} """ compute_node.run_cmd(cmd.format(bridge=bridge)) - - mac = compute_node.run_cmd("for vm in $(sudo virsh list | " - "grep running | awk '{print $2}'); " - "do echo -n ; sudo virsh dumpxml $vm| " - "grep -oP '52:54:[\da-f:]+' ;done") - compute_node.run_cmd("sudo virsh detach-interface --domain %s" - " --type bridge --mac %s" - % (libvirt_instance_name, mac)) -- cgit 1.2.3-korg