From 6d06b848796d68ef6677bb27b1e6e4bfa0762f80 Mon Sep 17 00:00:00 2001 From: Anthony Xu Date: Mon, 20 Aug 2012 17:54:17 -0700 Subject: [PATCH] CS-15921 : in 2.1 timeframe, -untagged string is appended to the end of instance name, in cleanup_rules function, we need to convert chain name to vm name correclty --- scripts/vm/hypervisor/xenserver/vmops | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/scripts/vm/hypervisor/xenserver/vmops b/scripts/vm/hypervisor/xenserver/vmops index 6ed12ca14f4..25916fef042 100755 --- a/scripts/vm/hypervisor/xenserver/vmops +++ b/scripts/vm/hypervisor/xenserver/vmops @@ -1052,16 +1052,18 @@ def cleanup_rules(session, args): cleaned = 0 cleanup = [] for chain in vmchains: - vm = session.xenapi.VM.get_by_name_label(chain) + vmname = chain + vm = session.xenapi.VM.get_by_name_label(vmname) if len(vm) != 1: - vm = session.xenapi.VM.get_by_name_label(chain + "-untagged") + vmname = chain + "-untagged" + vm = session.xenapi.VM.get_by_name_label(vmname) if len(vm) != 1: util.SMlog("chain " + chain + " does not correspond to a vm, cleaning up") - cleanup.append(chain) + cleanup.append(vmname) continue - if chain not in resident_vms: - util.SMlog("vm " + chain + " is not running, cleaning up") - cleanup.append(chain) + if vmname not in resident_vms: + util.SMlog("vm " + vmname + " is not running on this host, cleaning up") + cleanup.append(vmname) for vm_name in cleanup: destroy_network_rules_for_vm(session, {'vmName':vm_name})