]> git.apps.os.sepia.ceph.com Git - teuthology.git/commitdiff
Fix email scheduling bug 634/head
authorZack Cerza <zack@redhat.com>
Mon, 21 Sep 2015 19:41:23 +0000 (13:41 -0600)
committerZack Cerza <zack@redhat.com>
Mon, 21 Sep 2015 19:41:23 +0000 (13:41 -0600)
The call to teuthology_schedule was buried inside an 'if dry_run:'
clause. That clause is unnecessary since teuthology_schedule handles
dry_run cases - we pass it the same value as an arg

Signed-off-by: Zack Cerza <zack@redhat.com>
teuthology/suite.py

index 2f61d8153027a5377484c1beede3bee9a125c9f4..df0ba60f8cec45dd06febb6491f89a8f34479f69 100644 (file)
@@ -317,14 +317,12 @@ def prepare_and_schedule(job_config, suite_repo_path, base_yaml_paths, limit,
         arg.extend(['--email', job_config.email])
         if timeout:
             arg.extend(['--timeout', timeout])
-        if dry_run:
-            log.info('dry-run: %s' % ' '.join(arg))
-            teuthology_schedule(
-                args=arg,
-                dry_run=dry_run,
-                verbose=verbose,
-                log_prefix="Results email: ",
-            )
+        teuthology_schedule(
+            args=arg,
+            dry_run=dry_run,
+            verbose=verbose,
+            log_prefix="Results email: ",
+        )
         results_url = get_results_url(job_config.name)
         if results_url:
             log.info("Test results viewable at %s", results_url)