suite: make email-on-success the default behavior

This way you can tell when a run is complete, instead of wondering if
it's stuck in the queue.

diff --git a/teuthology/queue.py b/teuthology/queue.py
index 3467f03147a29f195c62ed23422c6da6f9ac1f02..aec52453b84b2fabcec71548c237ba33b35d0d1c 100644 (file)
--- a/teuthology/queue.py
+++ b/teuthology/queue.py
@@ -90,8 +90,6 @@ describe. One job is run at a time.
                 '--name',
                 job_config['name'],
                 ]
-            if job_config.get('email_on_success', False):
-                args.append('--email-on-success')
             subprocess.Popen(args=args)
         else:
             log.debug('Creating archive dir...')

diff --git a/teuthology/run.py b/teuthology/run.py
index 62b2e855f8822baaa897292ffa57fbfd5b3ca7a0..21d21e6dc6f87a6eb2180c70251ba21c4019ce70 100644 (file)
--- a/teuthology/run.py
+++ b/teuthology/run.py
@@ -208,12 +208,6 @@ def schedule():
         '--email',
         help='where to send the results of a suite (only applies to the last job in a suite)',
         )
-    parser.add_argument(
-        '--email-on-success',
-        action='store_true',
-        default=False,
-        help='email even if all tests pass',
-        )
     parser.add_argument(
         '--timeout',
         help='how many seconds to wait for jobs to finish before emailing results (only applies to the last job in a suite',
@@ -245,7 +239,6 @@ def schedule():
     if not ctx.last_in_suite:
         assert not ctx.email, '--email is only applicable to the last job in a suite'
         assert not ctx.timeout, '--timeout is only applicable to the last job in a suite'
-        assert not ctx.email_on_success, '--email-on-success is only applicable to the last job in a suite'
 
     from teuthology.misc import read_config, get_user
     if ctx.owner is None:
@@ -271,7 +264,6 @@ def schedule():
             name=ctx.name,
             last_in_suite=ctx.last_in_suite,
             email=ctx.email,
-            email_on_success=ctx.email_on_success,
             description=ctx.description,
             owner=ctx.owner,
             verbose=ctx.verbose,

diff --git a/teuthology/suite.py b/teuthology/suite.py
index e23acbaa62b674ff382f1266f5988fe11b686bf8..c9f69d9bdda886d15f112fda248d5f7bff607a6a 100644 (file)
--- a/teuthology/suite.py
+++ b/teuthology/suite.py
@@ -55,12 +55,6 @@ combination, and will override anything in the suite.
         '--email',
         help='address to email test failures to',
         )
-    parser.add_argument(
-        '--email-on-success',
-        action='store_true',
-        default=False,
-        help='email even if all tests pass',
-        )
     parser.add_argument(
         '--timeout',
         help='how many seconds to wait for jobs to finish before emailing results',
@@ -194,12 +188,6 @@ def results():
         '--email',
         help='address to email test failures to',
         )
-    parser.add_argument(
-        '--email-on-success',
-        action='store_true',
-        default=False,
-        help='email even if all tests pass',
-        )
     parser.add_argument(
         '--timeout',
         help='how many seconds to wait for all tests to finish (default no wait)',
@@ -299,7 +287,7 @@ def results():
                         reason=summary['failure_reason'],
                         ))
 
-    if not args.email or not (failures or unfinished or args.email_on_success):
+    if not args.email:
         return
 
     if failures or unfinished: