Clarify error when --lock is used with no targets

http://tracker.ceph.com/issues/11812

Signed-off-by: Zack Cerza <zack@redhat.com>

diff --git a/teuthology/lock.py b/teuthology/lock.py
index 48d295151a8788d34f4d5c21c7e1c212dec4a4bd..3a078fab501fbf527dfbef5aa05e6e5145bafe1c 100644 (file)
--- a/teuthology/lock.py
+++ b/teuthology/lock.py
@@ -182,9 +182,13 @@ def main(ctx):
             or ctx.update or ctx.brief, \
             'machines cannot be specified with that operation'
     else:
-        assert ctx.num_to_lock or ctx.list or ctx.list_targets or \
-            ctx.summary or ctx.brief, \
-            'machines must be specified for that operation'
+        if ctx.lock:
+            log.error("--lock requires specific machines passed as arguments")
+        else:
+            # This condition might never be hit, but it's not clear.
+            assert ctx.num_to_lock or ctx.list or ctx.list_targets or \
+                ctx.summary or ctx.brief, \
+                'machines must be specified for that operation'
     if ctx.all:
         assert ctx.list or ctx.list_targets or ctx.brief, \
             '--all can only be used with --list, --list-targets, and --brief'