Set ctx.cluster even when a job has no roles

This is to allow for teuthology to run self-tests that don't use
resources.

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

diff --git a/teuthology/run.py b/teuthology/run.py
index e1f79a1270969df7fc79214639ed02bc360ec32b..4f4646fd10fb2dfa190b15df04dfee357c4528bd 100644 (file)
--- a/teuthology/run.py
+++ b/teuthology/run.py
@@ -184,13 +184,15 @@ def get_initial_tasks(lock, config, machine_type):
         init_tasks.append({'internal.lock_machines': (
             len(config['roles']), machine_type)})
 
-
     init_tasks.append({'internal.save_config': None})
 
+    if 'roles' in config:
+        init_tasks.append({'internal.check_lock': None})
+
+    init_tasks.append({'internal.add_remotes': None})
+
     if 'roles' in config:
         init_tasks.extend([
-            {'internal.check_lock': None},
-            {'internal.add_remotes': None},
             {'console_log': None},
             {'internal.connect': None},
             {'internal.push_inventory': None},

diff --git a/teuthology/task/internal.py b/teuthology/task/internal.py
index 742f6fd078f25af94dda061ebaba80f476d0f8bb..367796127834e5e1550635d5fa8fdd6fc7a0aae6 100644 (file)
--- a/teuthology/task/internal.py
+++ b/teuthology/task/internal.py
@@ -308,6 +308,10 @@ def add_remotes(ctx, config):
     """
     Create a ctx.cluster object populated with remotes mapped to roles
     """
+    ctx.cluster = cluster.Cluster()
+    # Allow jobs to run without using nodes, for self-testing
+    if 'roles' not in ctx.config and 'targets' not in ctx.config:
+        return
     remotes = []
     machs = []
     for name in ctx.config['targets'].iterkeys():
@@ -321,7 +325,6 @@ def add_remotes(ctx, config):
             pass
         rem = remote.Remote(name=t, host_key=key, keep_alive=True)
         remotes.append(rem)
-    ctx.cluster = cluster.Cluster()
     if 'roles' in ctx.config:
         for rem, roles in zip(remotes, ctx.config['roles']):
             assert all(isinstance(role, str) for role in roles), \