Skip to content

Commit

Permalink
Don't unlock VMs when destroy fails
Browse files Browse the repository at this point in the history
Signed-off-by: Zack Cerza <zack@redhat.com>
  • Loading branch information
zmc committed Feb 24, 2017
1 parent bf25efb commit 77185fd
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions teuthology/lock/ops.py
Expand Up @@ -163,6 +163,7 @@ def unlock_one(ctx, name, user, description=None):
name = misc.canonicalize_hostname(name, user=None)
if not teuthology.provision.destroy_if_vm(ctx, name, user, description):
log.error('destroy failed for %s', name)
return False
request = dict(name=name, locked=False, locked_by=user,
description=description)
uri = os.path.join(config.lock_server, 'nodes', name, 'lock', '')
Expand Down

0 comments on commit 77185fd

Please sign in to comment.