diff options
author | ncteisen <ncteisen@gmail.com> | 2017-12-11 18:00:40 -0800 |
---|---|---|
committer | ncteisen <ncteisen@gmail.com> | 2017-12-11 18:08:35 -0800 |
commit | 888093c6ed0d24eed699173b5fb35235fe7a6069 (patch) | |
tree | ffbbc9fbfaacff17c3b94a2f2d780ece6abbfa36 /tools/run_tests/task_runner.py | |
parent | a69c6901f9c4e375ad543cb4ee6ce747d953bf18 (diff) |
yapf run_tests
Diffstat (limited to 'tools/run_tests/task_runner.py')
-rwxr-xr-x | tools/run_tests/task_runner.py | 100 |
1 files changed, 50 insertions, 50 deletions
diff --git a/tools/run_tests/task_runner.py b/tools/run_tests/task_runner.py index a065bb84cb..794db6e1be 100755 --- a/tools/run_tests/task_runner.py +++ b/tools/run_tests/task_runner.py @@ -12,7 +12,6 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. - """Runs selected gRPC test/build tasks.""" from __future__ import print_function @@ -32,52 +31,54 @@ _TARGETS += artifact_targets.targets() _TARGETS += distribtest_targets.targets() _TARGETS += package_targets.targets() + def _create_build_map(): - """Maps task names and labels to list of tasks to be built.""" - target_build_map = dict([(target.name, [target]) - for target in _TARGETS]) - if len(_TARGETS) > len(target_build_map.keys()): - raise Exception('Target names need to be unique') - - label_build_map = {} - label_build_map['all'] = [t for t in _TARGETS] # to build all targets - for target in _TARGETS: - for label in target.labels: - if label in label_build_map: - label_build_map[label].append(target) - else: - label_build_map[label] = [target] - - if set(target_build_map.keys()).intersection(label_build_map.keys()): - raise Exception('Target names need to be distinct from label names') - return dict( target_build_map.items() + label_build_map.items()) + """Maps task names and labels to list of tasks to be built.""" + target_build_map = dict([(target.name, [target]) for target in _TARGETS]) + if len(_TARGETS) > len(target_build_map.keys()): + raise Exception('Target names need to be unique') + + label_build_map = {} + label_build_map['all'] = [t for t in _TARGETS] # to build all targets + for target in _TARGETS: + for label in target.labels: + if label in label_build_map: + label_build_map[label].append(target) + else: + label_build_map[label] = [target] + + if set(target_build_map.keys()).intersection(label_build_map.keys()): + raise Exception('Target names need to be distinct from label names') + return dict(target_build_map.items() + label_build_map.items()) _BUILD_MAP = _create_build_map() argp = argparse.ArgumentParser(description='Runs build/test targets.') -argp.add_argument('-b', '--build', - choices=sorted(_BUILD_MAP.keys()), - nargs='+', - default=['all'], - help='Target name or target label to build.') -argp.add_argument('-f', '--filter', - choices=sorted(_BUILD_MAP.keys()), - nargs='+', - default=[], - help='Filter targets to build with AND semantics.') +argp.add_argument( + '-b', + '--build', + choices=sorted(_BUILD_MAP.keys()), + nargs='+', + default=['all'], + help='Target name or target label to build.') +argp.add_argument( + '-f', + '--filter', + choices=sorted(_BUILD_MAP.keys()), + nargs='+', + default=[], + help='Filter targets to build with AND semantics.') argp.add_argument('-j', '--jobs', default=multiprocessing.cpu_count(), type=int) -argp.add_argument('-t', '--travis', - default=False, - action='store_const', - const=True) +argp.add_argument( + '-t', '--travis', default=False, action='store_const', const=True) args = argp.parse_args() # Figure out which targets to build targets = [] for label in args.build: - targets += _BUILD_MAP[label] + targets += _BUILD_MAP[label] # Among targets selected by -b, filter out those that don't match the filter targets = [t for t in targets if all(f in t.labels for f in args.filter)] @@ -86,30 +87,29 @@ targets = sorted(set(targets)) # Execute pre-build phase prebuild_jobs = [] for target in targets: - prebuild_jobs += target.pre_build_jobspecs() + prebuild_jobs += target.pre_build_jobspecs() if prebuild_jobs: - num_failures, _ = jobset.run( - prebuild_jobs, newline_on_success=True, maxjobs=args.jobs) - if num_failures != 0: - jobset.message('FAILED', 'Pre-build phase failed.', do_newline=True) - sys.exit(1) + num_failures, _ = jobset.run( + prebuild_jobs, newline_on_success=True, maxjobs=args.jobs) + if num_failures != 0: + jobset.message('FAILED', 'Pre-build phase failed.', do_newline=True) + sys.exit(1) build_jobs = [] for target in targets: - build_jobs.append(target.build_jobspec()) + build_jobs.append(target.build_jobspec()) if not build_jobs: - print('Nothing to build.') - sys.exit(1) + print('Nothing to build.') + sys.exit(1) jobset.message('START', 'Building targets.', do_newline=True) num_failures, resultset = jobset.run( build_jobs, newline_on_success=True, maxjobs=args.jobs) -report_utils.render_junit_xml_report(resultset, 'report_taskrunner_sponge_log.xml', - suite_name='tasks') +report_utils.render_junit_xml_report( + resultset, 'report_taskrunner_sponge_log.xml', suite_name='tasks') if num_failures == 0: - jobset.message('SUCCESS', 'All targets built successfully.', - do_newline=True) + jobset.message( + 'SUCCESS', 'All targets built successfully.', do_newline=True) else: - jobset.message('FAILED', 'Failed to build targets.', - do_newline=True) - sys.exit(1) + jobset.message('FAILED', 'Failed to build targets.', do_newline=True) + sys.exit(1) |