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