jjb/jenkins-job-builder
Code Issues Proposed changes

Tweak jenkins-jobs to conform with pep8 (v1.2)

Browse Source 
Change-Id: I91ecac2e8d2d6b6b58f9a7fccc91897270ffe44d
Reviewed-on: https://review.openstack.org/13267
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
This commit is contained in:
Manuel Desbonnet 2012-09-19 09:49:36 +01:00 committed by Jenkins
parent d4e443e632
commit a760c88433
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
jenkins-jobs
View File

@ -5,6 +5,7 @@ import argparse
import ConfigParser
import logging
def main():
parser = argparse.ArgumentParser()
subparser = parser.add_subparsers(help='update, test or delete job',
@ -14,7 +15,8 @@ def main():
parser_update.add_argument('name', help='name of job', nargs='?')
parser_test = subparser.add_parser('test')
parser_test.add_argument('path', help='Path to YAML file or directory')
parser_test.add_argument('-o', dest='output_dir', help='Path to output XML')
parser_test.add_argument('-o', dest='output_dir',
help='Path to output XML')
parser_test.add_argument('name', help='name of job', nargs='?')
parser_delete = subparser.add_parser('delete')
parser_delete.add_argument('name', help='name of job')
@ -41,9 +43,9 @@ def main():
else:
config = {}
logger.debug("Config: {0}".format(config))
builder = jenkins_jobs.builder.Builder(config.get('jenkins','url'),
config.get('jenkins','user'),
config.get('jenkins','password'))
builder = jenkins_jobs.builder.Builder(config.get('jenkins', 'url'),
config.get('jenkins', 'user'),
config.get('jenkins', 'password'))
if options.command == 'delete':
logger.info("Deleting job {0}".format(options.name))