From a760c884333d234e23308cc86805b59587251729 Mon Sep 17 00:00:00 2001 From: Manuel Desbonnet Date: Wed, 19 Sep 2012 09:49:36 +0100 Subject: [PATCH] Tweak jenkins-jobs to conform with pep8 (v1.2) Change-Id: I91ecac2e8d2d6b6b58f9a7fccc91897270ffe44d Reviewed-on: https://review.openstack.org/13267 Reviewed-by: James E. Blair Approved: Clark Boylan Reviewed-by: Clark Boylan Tested-by: Jenkins --- jenkins-jobs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/jenkins-jobs b/jenkins-jobs index 77fe1fd13..d3e8cbf47 100755 --- a/jenkins-jobs +++ b/jenkins-jobs @@ -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))