From 4d5773ffa0c23cb99095be515d7968a305836988 Mon Sep 17 00:00:00 2001 From: Zack Cerza Date: Mon, 21 Sep 2015 13:41:23 -0600 Subject: [PATCH] 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 --- teuthology/suite.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) 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) -- 2.47.3