Improve unlock error messages.

Added messages if the hostname is invalid, and if
the user is not the owner of the lock.

Fixes: 6295
Signed-off-by: Warren Usui <warren.usui@inktank.com>

diff --git a/teuthology/lock.py b/teuthology/lock.py
index 589dfabfb21c16c33282e52d7af1b57d60c7984f..42aa01d93f6d27577ba27f6d879ffa852c239191 100644 (file)
--- a/teuthology/lock.py
+++ b/teuthology/lock.py
@@ -73,7 +73,7 @@ def lock_one(ctx, name, user=None, description=None):
 def unlock_one(ctx, name, user=None):
     if user is None:
         user = misc.get_user()
-    success, _, _ = ls.send_request(
+    success, _, http_ret = ls.send_request(
         'DELETE',
         config.lock_server + '/' + name + '?' +
         urllib.urlencode(dict(user=user)))
@@ -84,6 +84,10 @@ def unlock_one(ctx, name, user=None):
             log.info('%s is not locked' % name)
     else:
         log.error('failed to unlock %s', name)
+        failure_types = {403: 'You do not have %s locked',
+                         404: '%s is an invalid host name'}
+        if http_ret in failure_types:
+            log.error(failure_types[http_ret], name)
     return success