From: Zack Cerza <zack@redhat.com>
Date: Mon, 21 Sep 2015 19:41:23 +0000 (-0600)
Subject: Fix email scheduling bug
X-Git-Tag: 1.1.0~813^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F634%2Fhead;p=teuthology.git

Fix email scheduling bug

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>
---

diff --git a/teuthology/suite.py b/teuthology/suite.py
index 2f61d8153..df0ba60f8 100644
--- a/teuthology/suite.py
+++ b/teuthology/suite.py
@@ -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)