[Qa-jenkins-scm] [jenkins.debian.net] 02/03: reproducible: remote_scheduler: don't require the -s option at argparse level, bug check it later on (this allow passing just --null via argv)

[Holger Levsen]

This is an automated email from the git hooks/post-receive script.

holger pushed a commit to branch master
in repository jenkins.debian.net.

commit 3dfdc40bf58e17b33e384f5f73d110953d659285
Author: Mattia Rizzolo <mattia at mapreri.org>
Date:   Wed Aug 19 21:11:04 2015 +0000

    reproducible: remote_scheduler: don't require the -s option at argparse level, bug check it later on (this allow passing just --null via argv)
---
 bin/reproducible_remote_scheduler.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/bin/reproducible_remote_scheduler.py b/bin/reproducible_remote_scheduler.py
index d2de98b..7faab16 100755
--- a/bin/reproducible_remote_scheduler.py
+++ b/bin/reproducible_remote_scheduler.py
@@ -51,7 +51,7 @@ parser.add_argument('-b', '--before', required=False,
 parser.add_argument('-a', '--architecture', required=False, default='amd64',
                     help='Specify the architecture to schedule for ' +
                     '(defaults to amd64)')
-parser.add_argument('-s', '--suite', required=True,
+parser.add_argument('-s', '--suite', required=False,
                     help='Specify the suite to schedule in')
 parser.add_argument('packages', metavar='package', nargs='*',
                     help='list of packages to reschedule')
@@ -105,6 +105,10 @@ log.debug('Suite: ' + suite)
 log.debug('Architecture: ' + arch)
 log.debug('Packages: ' + ' '.join(packages))
 
+if not suite:
+    log.critical('You need to specify the suite name')
+    sys.exit(1)
+
 if suite not in SUITES:
     log.critical('The specified suite is not being tested.')
     log.critical('Please choose between ' + ', '.join(SUITES))

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/qa/jenkins.debian.net.git