Siddharth Shukla | 8e64d90 | 2017-03-12 19:50:18 +0100 | [diff] [blame] | 1 | #!/usr/bin/env python |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 2 | # Copyright 2015 gRPC authors. |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 3 | # |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 4 | # Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | # you may not use this file except in compliance with the License. |
| 6 | # You may obtain a copy of the License at |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 7 | # |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 8 | # http://www.apache.org/licenses/LICENSE-2.0 |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 9 | # |
Jan Tattermusch | 7897ae9 | 2017-06-07 22:57:36 +0200 | [diff] [blame] | 10 | # Unless required by applicable law or agreed to in writing, software |
| 11 | # distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | # See the License for the specific language governing permissions and |
| 14 | # limitations under the License. |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 15 | |
| 16 | """Run test matrix.""" |
| 17 | |
Siddharth Shukla | d194f59 | 2017-03-11 19:12:43 +0100 | [diff] [blame] | 18 | from __future__ import print_function |
| 19 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 20 | import argparse |
Jan Tattermusch | a1906d5 | 2016-09-19 18:37:17 +0200 | [diff] [blame] | 21 | import multiprocessing |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 22 | import os |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 23 | import sys |
Jan Tattermusch | 5c79a31 | 2016-12-20 11:02:50 +0100 | [diff] [blame] | 24 | |
| 25 | import python_utils.jobset as jobset |
| 26 | import python_utils.report_utils as report_utils |
| 27 | from python_utils.filter_pull_request_tests import filter_tests |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 28 | |
| 29 | _ROOT = os.path.abspath(os.path.join(os.path.dirname(sys.argv[0]), '../..')) |
| 30 | os.chdir(_ROOT) |
| 31 | |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 32 | _DEFAULT_RUNTESTS_TIMEOUT = 1*60*60 |
| 33 | |
Jan Tattermusch | 060eb87 | 2016-09-20 16:06:13 +0200 | [diff] [blame] | 34 | # Set the timeout high to allow enough time for sanitizers and pre-building |
| 35 | # clang docker. |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 36 | _CPP_RUNTESTS_TIMEOUT = 4*60*60 |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 37 | |
Jan Tattermusch | 67cd21e | 2017-09-19 16:21:20 +0200 | [diff] [blame^] | 38 | # C++ TSAN takes longer than other sanitizers |
| 39 | _CPP_TSAN_RUNTESTS_TIMEOUT = 8*60*60 |
| 40 | |
Jan Tattermusch | a1906d5 | 2016-09-19 18:37:17 +0200 | [diff] [blame] | 41 | # Number of jobs assigned to each run_tests.py instance |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 42 | _DEFAULT_INNER_JOBS = 2 |
Jan Tattermusch | a1906d5 | 2016-09-19 18:37:17 +0200 | [diff] [blame] | 43 | |
Jan Tattermusch | 5a59c43 | 2017-03-07 19:57:13 +0100 | [diff] [blame] | 44 | # report suffix is important for reports to get picked up by internal CI |
| 45 | _REPORT_SUFFIX = 'sponge_log.xml' |
| 46 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 47 | |
Jan Tattermusch | ac9c9f9 | 2017-04-28 20:56:05 +0200 | [diff] [blame] | 48 | def _report_filename(name): |
| 49 | """Generates report file name""" |
| 50 | return 'report_%s_%s' % (name, _REPORT_SUFFIX) |
| 51 | |
| 52 | |
| 53 | def _report_filename_internal_ci(name): |
| 54 | """Generates report file name that leads to better presentation by internal CI""" |
| 55 | return '%s/%s' % (name, _REPORT_SUFFIX) |
| 56 | |
| 57 | |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 58 | def _docker_jobspec(name, runtests_args=[], runtests_envs={}, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 59 | inner_jobs=_DEFAULT_INNER_JOBS, |
| 60 | timeout_seconds=None): |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 61 | """Run a single instance of run_tests.py in a docker container""" |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 62 | if not timeout_seconds: |
| 63 | timeout_seconds = _DEFAULT_RUNTESTS_TIMEOUT |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 64 | test_job = jobset.JobSpec( |
| 65 | cmdline=['python', 'tools/run_tests/run_tests.py', |
| 66 | '--use_docker', |
| 67 | '-t', |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 68 | '-j', str(inner_jobs), |
Jan Tattermusch | ac9c9f9 | 2017-04-28 20:56:05 +0200 | [diff] [blame] | 69 | '-x', _report_filename(name), |
Jan Tattermusch | cfcc075 | 2016-10-09 17:02:34 +0200 | [diff] [blame] | 70 | '--report_suite_name', '%s' % name] + runtests_args, |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 71 | environ=runtests_envs, |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 72 | shortname='run_tests_%s' % name, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 73 | timeout_seconds=timeout_seconds) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 74 | return test_job |
| 75 | |
| 76 | |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 77 | def _workspace_jobspec(name, runtests_args=[], workspace_name=None, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 78 | runtests_envs={}, inner_jobs=_DEFAULT_INNER_JOBS, |
| 79 | timeout_seconds=None): |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 80 | """Run a single instance of run_tests.py in a separate workspace""" |
Jan Tattermusch | a1906d5 | 2016-09-19 18:37:17 +0200 | [diff] [blame] | 81 | if not workspace_name: |
| 82 | workspace_name = 'workspace_%s' % name |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 83 | if not timeout_seconds: |
| 84 | timeout_seconds = _DEFAULT_RUNTESTS_TIMEOUT |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 85 | env = {'WORKSPACE_NAME': workspace_name} |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 86 | env.update(runtests_envs) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 87 | test_job = jobset.JobSpec( |
Jan Tattermusch | ff61b8c | 2017-03-01 15:57:33 +0100 | [diff] [blame] | 88 | cmdline=['bash', |
| 89 | 'tools/run_tests/helper_scripts/run_tests_in_workspace.sh', |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 90 | '-t', |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 91 | '-j', str(inner_jobs), |
Jan Tattermusch | ac9c9f9 | 2017-04-28 20:56:05 +0200 | [diff] [blame] | 92 | '-x', '../%s' % _report_filename(name), |
Jan Tattermusch | cfcc075 | 2016-10-09 17:02:34 +0200 | [diff] [blame] | 93 | '--report_suite_name', '%s' % name] + runtests_args, |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 94 | environ=env, |
| 95 | shortname='run_tests_%s' % name, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 96 | timeout_seconds=timeout_seconds) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 97 | return test_job |
| 98 | |
| 99 | |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 100 | def _generate_jobs(languages, configs, platforms, iomgr_platform = 'native', |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 101 | arch=None, compiler=None, |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 102 | labels=[], extra_args=[], extra_envs={}, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 103 | inner_jobs=_DEFAULT_INNER_JOBS, |
| 104 | timeout_seconds=None): |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 105 | result = [] |
| 106 | for language in languages: |
| 107 | for platform in platforms: |
| 108 | for config in configs: |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 109 | name = '%s_%s_%s_%s' % (language, platform, config, iomgr_platform) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 110 | runtests_args = ['-l', language, |
murgatroid99 | d0cda5c | 2017-03-07 18:04:52 -0800 | [diff] [blame] | 111 | '-c', config, |
| 112 | '--iomgr_platform', iomgr_platform] |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 113 | if arch or compiler: |
| 114 | name += '_%s_%s' % (arch, compiler) |
| 115 | runtests_args += ['--arch', arch, |
| 116 | '--compiler', compiler] |
Jan Tattermusch | 0b0d9d4 | 2017-06-13 09:55:59 +0200 | [diff] [blame] | 117 | if '--build_only' in extra_args: |
| 118 | name += '_buildonly' |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 119 | for extra_env in extra_envs: |
Yuchen Zeng | 87b5910 | 2016-11-08 10:50:12 -0800 | [diff] [blame] | 120 | name += '_%s_%s' % (extra_env, extra_envs[extra_env]) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 121 | |
Jan Tattermusch | 6d7c6ef | 2016-09-22 13:40:48 +0200 | [diff] [blame] | 122 | runtests_args += extra_args |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 123 | if platform == 'linux': |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 124 | job = _docker_jobspec(name=name, runtests_args=runtests_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 125 | runtests_envs=extra_envs, inner_jobs=inner_jobs, |
| 126 | timeout_seconds=timeout_seconds) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 127 | else: |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 128 | job = _workspace_jobspec(name=name, runtests_args=runtests_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 129 | runtests_envs=extra_envs, inner_jobs=inner_jobs, |
| 130 | timeout_seconds=timeout_seconds) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 131 | |
murgatroid99 | d0cda5c | 2017-03-07 18:04:52 -0800 | [diff] [blame] | 132 | job.labels = [platform, config, language, iomgr_platform] + labels |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 133 | result.append(job) |
| 134 | return result |
| 135 | |
| 136 | |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 137 | def _create_test_jobs(extra_args=[], inner_jobs=_DEFAULT_INNER_JOBS): |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 138 | test_jobs = [] |
| 139 | # supported on linux only |
| 140 | test_jobs += _generate_jobs(languages=['sanity', 'php7'], |
| 141 | configs=['dbg', 'opt'], |
| 142 | platforms=['linux'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 143 | labels=['basictests', 'multilang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 144 | extra_args=extra_args, |
| 145 | inner_jobs=inner_jobs) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 146 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 147 | # supported on all platforms. |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 148 | test_jobs += _generate_jobs(languages=['c'], |
Jan Tattermusch | 6d7c6ef | 2016-09-22 13:40:48 +0200 | [diff] [blame] | 149 | configs=['dbg', 'opt'], |
| 150 | platforms=['linux', 'macos', 'windows'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 151 | labels=['basictests', 'corelang'], |
| 152 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 153 | inner_jobs=inner_jobs, |
| 154 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 155 | |
| 156 | test_jobs += _generate_jobs(languages=['csharp', 'node', 'python'], |
| 157 | configs=['dbg', 'opt'], |
| 158 | platforms=['linux', 'macos', 'windows'], |
| 159 | labels=['basictests', 'multilang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 160 | extra_args=extra_args, |
| 161 | inner_jobs=inner_jobs) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 162 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 163 | # supported on linux and mac. |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 164 | test_jobs += _generate_jobs(languages=['c++'], |
Jan Tattermusch | 6d7c6ef | 2016-09-22 13:40:48 +0200 | [diff] [blame] | 165 | configs=['dbg', 'opt'], |
| 166 | platforms=['linux', 'macos'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 167 | labels=['basictests', 'corelang'], |
| 168 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 169 | inner_jobs=inner_jobs, |
| 170 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 171 | |
| 172 | test_jobs += _generate_jobs(languages=['ruby', 'php'], |
| 173 | configs=['dbg', 'opt'], |
| 174 | platforms=['linux', 'macos'], |
| 175 | labels=['basictests', 'multilang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 176 | extra_args=extra_args, |
| 177 | inner_jobs=inner_jobs) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 178 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 179 | # supported on mac only. |
| 180 | test_jobs += _generate_jobs(languages=['objc'], |
| 181 | configs=['dbg', 'opt'], |
| 182 | platforms=['macos'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 183 | labels=['basictests', 'multilang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 184 | extra_args=extra_args, |
| 185 | inner_jobs=inner_jobs) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 186 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 187 | # sanitizers |
| 188 | test_jobs += _generate_jobs(languages=['c'], |
Matt Kwong | d42f55b | 2017-05-04 15:29:32 -0700 | [diff] [blame] | 189 | configs=['msan', 'asan', 'tsan', 'ubsan'], |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 190 | platforms=['linux'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 191 | labels=['sanitizers', 'corelang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 192 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 193 | inner_jobs=inner_jobs, |
| 194 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 195 | test_jobs += _generate_jobs(languages=['c++'], |
Jan Tattermusch | 67cd21e | 2017-09-19 16:21:20 +0200 | [diff] [blame^] | 196 | configs=['asan'], |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 197 | platforms=['linux'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 198 | labels=['sanitizers', 'corelang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 199 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 200 | inner_jobs=inner_jobs, |
| 201 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
Jan Tattermusch | 67cd21e | 2017-09-19 16:21:20 +0200 | [diff] [blame^] | 202 | test_jobs += _generate_jobs(languages=['c++'], |
| 203 | configs=['tsan'], |
| 204 | platforms=['linux'], |
| 205 | labels=['sanitizers', 'corelang'], |
| 206 | extra_args=extra_args, |
| 207 | inner_jobs=inner_jobs, |
| 208 | timeout_seconds=_CPP_TSAN_RUNTESTS_TIMEOUT) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 209 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 210 | return test_jobs |
| 211 | |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 212 | |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 213 | def _create_portability_test_jobs(extra_args=[], inner_jobs=_DEFAULT_INNER_JOBS): |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 214 | test_jobs = [] |
| 215 | # portability C x86 |
| 216 | test_jobs += _generate_jobs(languages=['c'], |
| 217 | configs=['dbg'], |
| 218 | platforms=['linux'], |
| 219 | arch='x86', |
| 220 | compiler='default', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 221 | labels=['portability', 'corelang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 222 | extra_args=extra_args, |
| 223 | inner_jobs=inner_jobs) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 224 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 225 | # portability C and C++ on x64 |
Ken Payson | 0290906 | 2017-05-04 12:33:58 -0700 | [diff] [blame] | 226 | for compiler in ['gcc4.8', 'gcc5.3', 'gcc_musl', |
Matt Kwong | e3beac9 | 2016-11-01 12:53:04 -0700 | [diff] [blame] | 227 | 'clang3.5', 'clang3.6', 'clang3.7']: |
Vijay Pai | add2e4d | 2017-05-10 10:25:13 -0700 | [diff] [blame] | 228 | test_jobs += _generate_jobs(languages=['c', 'c++'], |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 229 | configs=['dbg'], |
| 230 | platforms=['linux'], |
| 231 | arch='x64', |
| 232 | compiler=compiler, |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 233 | labels=['portability', 'corelang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 234 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 235 | inner_jobs=inner_jobs, |
| 236 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 237 | |
Jan Tattermusch | a8003e8 | 2017-08-23 15:43:17 +0200 | [diff] [blame] | 238 | # portability C on Windows 64-bit (x86 is the default) |
| 239 | test_jobs += _generate_jobs(languages=['c'], |
| 240 | configs=['dbg'], |
| 241 | platforms=['windows'], |
| 242 | arch='x64', |
| 243 | compiler='default', |
| 244 | labels=['portability', 'corelang'], |
| 245 | extra_args=extra_args, |
| 246 | inner_jobs=inner_jobs) |
Jan Tattermusch | add3225 | 2017-06-26 17:13:14 +0200 | [diff] [blame] | 247 | |
| 248 | # portability C++ on Windows |
| 249 | # TODO(jtattermusch): some of the tests are failing, so we force --build_only |
| 250 | test_jobs += _generate_jobs(languages=['c++'], |
| 251 | configs=['dbg'], |
| 252 | platforms=['windows'], |
| 253 | arch='default', |
| 254 | compiler='default', |
| 255 | labels=['portability', 'corelang'], |
| 256 | extra_args=extra_args + ['--build_only'], |
| 257 | inner_jobs=inner_jobs) |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 258 | |
Jan Tattermusch | a8003e8 | 2017-08-23 15:43:17 +0200 | [diff] [blame] | 259 | # portability C and C++ on Windows using VS2017 (build only) |
| 260 | # TODO(jtattermusch): some of the tests are failing, so we force --build_only |
| 261 | test_jobs += _generate_jobs(languages=['c', 'c++'], |
| 262 | configs=['dbg'], |
| 263 | platforms=['windows'], |
| 264 | arch='x64', |
| 265 | compiler='cmake_vs2017', |
| 266 | labels=['portability', 'corelang'], |
| 267 | extra_args=extra_args + ['--build_only'], |
| 268 | inner_jobs=inner_jobs) |
| 269 | |
Yuchen Zeng | 1561862 | 2017-03-19 22:31:14 -0700 | [diff] [blame] | 270 | # C and C++ with the c-ares DNS resolver on Linux |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 271 | test_jobs += _generate_jobs(languages=['c', 'c++'], |
Yuchen Zeng | 87b5910 | 2016-11-08 10:50:12 -0800 | [diff] [blame] | 272 | configs=['dbg'], platforms=['linux'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 273 | labels=['portability', 'corelang'], |
Yuchen Zeng | 87b5910 | 2016-11-08 10:50:12 -0800 | [diff] [blame] | 274 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 275 | extra_envs={'GRPC_DNS_RESOLVER': 'ares'}, |
| 276 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
Yuchen Zeng | 87b5910 | 2016-11-08 10:50:12 -0800 | [diff] [blame] | 277 | |
Yuchen Zeng | 1561862 | 2017-03-19 22:31:14 -0700 | [diff] [blame] | 278 | # TODO(zyc): Turn on this test after adding c-ares support on windows. |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 279 | # C with the c-ares DNS resolver on Windows |
Yuchen Zeng | 1561862 | 2017-03-19 22:31:14 -0700 | [diff] [blame] | 280 | # test_jobs += _generate_jobs(languages=['c'], |
| 281 | # configs=['dbg'], platforms=['windows'], |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 282 | # labels=['portability', 'corelang'], |
Yuchen Zeng | 1561862 | 2017-03-19 22:31:14 -0700 | [diff] [blame] | 283 | # extra_args=extra_args, |
| 284 | # extra_envs={'GRPC_DNS_RESOLVER': 'ares'}) |
Yuchen Zeng | fdae4bd | 2016-11-07 17:46:16 -0800 | [diff] [blame] | 285 | |
Jan Tattermusch | add3225 | 2017-06-26 17:13:14 +0200 | [diff] [blame] | 286 | # C and C++ build with cmake on Linux |
Jan Tattermusch | dfb03bb | 2017-01-25 19:27:58 +0100 | [diff] [blame] | 287 | # TODO(jtattermusch): some of the tests are failing, so we force --build_only |
| 288 | # to make sure it's buildable at least. |
| 289 | test_jobs += _generate_jobs(languages=['c', 'c++'], |
| 290 | configs=['dbg'], |
Jan Tattermusch | add3225 | 2017-06-26 17:13:14 +0200 | [diff] [blame] | 291 | platforms=['linux'], |
Jan Tattermusch | dfb03bb | 2017-01-25 19:27:58 +0100 | [diff] [blame] | 292 | arch='default', |
| 293 | compiler='cmake', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 294 | labels=['portability', 'corelang'], |
Jan Tattermusch | dfb03bb | 2017-01-25 19:27:58 +0100 | [diff] [blame] | 295 | extra_args=extra_args + ['--build_only'], |
| 296 | inner_jobs=inner_jobs) |
| 297 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 298 | test_jobs += _generate_jobs(languages=['python'], |
| 299 | configs=['dbg'], |
| 300 | platforms=['linux'], |
| 301 | arch='default', |
Ken Payson | 0290906 | 2017-05-04 12:33:58 -0700 | [diff] [blame] | 302 | compiler='python_alpine', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 303 | labels=['portability', 'multilang'], |
Ken Payson | 0290906 | 2017-05-04 12:33:58 -0700 | [diff] [blame] | 304 | extra_args=extra_args, |
| 305 | inner_jobs=inner_jobs) |
| 306 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 307 | test_jobs += _generate_jobs(languages=['csharp'], |
| 308 | configs=['dbg'], |
| 309 | platforms=['linux'], |
| 310 | arch='default', |
| 311 | compiler='coreclr', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 312 | labels=['portability', 'multilang'], |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 313 | extra_args=extra_args, |
| 314 | inner_jobs=inner_jobs) |
murgatroid99 | 804c9e9 | 2016-12-05 12:19:57 -0800 | [diff] [blame] | 315 | |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 316 | test_jobs += _generate_jobs(languages=['c'], |
| 317 | configs=['dbg'], |
| 318 | platforms=['linux'], |
| 319 | iomgr_platform='uv', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 320 | labels=['portability', 'corelang'], |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 321 | extra_args=extra_args, |
Jan Tattermusch | 29828c5 | 2017-09-12 16:42:55 +0200 | [diff] [blame] | 322 | inner_jobs=inner_jobs, |
| 323 | timeout_seconds=_CPP_RUNTESTS_TIMEOUT) |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 324 | |
murgatroid99 | 804c9e9 | 2016-12-05 12:19:57 -0800 | [diff] [blame] | 325 | test_jobs += _generate_jobs(languages=['node'], |
| 326 | configs=['dbg'], |
| 327 | platforms=['linux'], |
| 328 | arch='default', |
murgatroid99 | f25f505 | 2017-04-13 17:22:32 -0700 | [diff] [blame] | 329 | compiler='electron1.6', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 330 | labels=['portability', 'multilang'], |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 331 | extra_args=extra_args, |
| 332 | inner_jobs=inner_jobs) |
| 333 | |
| 334 | test_jobs += _generate_jobs(languages=['node'], |
| 335 | configs=['dbg'], |
| 336 | platforms=['linux'], |
| 337 | arch='default', |
| 338 | compiler='node4', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 339 | labels=['portability', 'multilang'], |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 340 | extra_args=extra_args, |
| 341 | inner_jobs=inner_jobs) |
| 342 | |
| 343 | test_jobs += _generate_jobs(languages=['node'], |
| 344 | configs=['dbg'], |
| 345 | platforms=['linux'], |
| 346 | arch='default', |
| 347 | compiler='node6', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 348 | labels=['portability', 'multilang'], |
murgatroid99 | 1191b72 | 2017-02-08 11:56:52 -0800 | [diff] [blame] | 349 | extra_args=extra_args, |
| 350 | inner_jobs=inner_jobs) |
| 351 | |
murgatroid99 | 4bea5b9 | 2017-05-30 17:37:02 -0700 | [diff] [blame] | 352 | test_jobs += _generate_jobs(languages=['node'], |
| 353 | configs=['dbg'], |
| 354 | platforms=['linux'], |
| 355 | arch='default', |
| 356 | compiler='node7', |
Jan Tattermusch | a389942 | 2017-07-13 11:24:02 +0200 | [diff] [blame] | 357 | labels=['portability', 'multilang'], |
murgatroid99 | 4bea5b9 | 2017-05-30 17:37:02 -0700 | [diff] [blame] | 358 | extra_args=extra_args, |
| 359 | inner_jobs=inner_jobs) |
| 360 | |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 361 | return test_jobs |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 362 | |
| 363 | |
Jan Tattermusch | 6d7c6ef | 2016-09-22 13:40:48 +0200 | [diff] [blame] | 364 | def _allowed_labels(): |
| 365 | """Returns a list of existing job labels.""" |
| 366 | all_labels = set() |
| 367 | for job in _create_test_jobs() + _create_portability_test_jobs(): |
| 368 | for label in job.labels: |
| 369 | all_labels.add(label) |
| 370 | return sorted(all_labels) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 371 | |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 372 | |
Jan Tattermusch | 68e27bf | 2016-12-16 14:09:03 +0100 | [diff] [blame] | 373 | def _runs_per_test_type(arg_str): |
Jan Tattermusch | 6a85129 | 2016-12-20 10:20:42 +0100 | [diff] [blame] | 374 | """Auxiliary function to parse the "runs_per_test" flag.""" |
| 375 | try: |
| 376 | n = int(arg_str) |
| 377 | if n <= 0: raise ValueError |
| 378 | return n |
| 379 | except: |
| 380 | msg = '\'{}\' is not a positive integer'.format(arg_str) |
| 381 | raise argparse.ArgumentTypeError(msg) |
Jan Tattermusch | 68e27bf | 2016-12-16 14:09:03 +0100 | [diff] [blame] | 382 | |
| 383 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 384 | if __name__ == "__main__": |
| 385 | argp = argparse.ArgumentParser(description='Run a matrix of run_tests.py tests.') |
| 386 | argp.add_argument('-j', '--jobs', |
| 387 | default=multiprocessing.cpu_count()/_DEFAULT_INNER_JOBS, |
| 388 | type=int, |
| 389 | help='Number of concurrent run_tests.py instances.') |
| 390 | argp.add_argument('-f', '--filter', |
| 391 | choices=_allowed_labels(), |
| 392 | nargs='+', |
| 393 | default=[], |
| 394 | help='Filter targets to run by label with AND semantics.') |
Jan Tattermusch | 6a85129 | 2016-12-20 10:20:42 +0100 | [diff] [blame] | 395 | argp.add_argument('--exclude', |
| 396 | choices=_allowed_labels(), |
| 397 | nargs='+', |
| 398 | default=[], |
| 399 | help='Exclude targets with any of given labels.') |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 400 | argp.add_argument('--build_only', |
| 401 | default=False, |
| 402 | action='store_const', |
| 403 | const=True, |
| 404 | help='Pass --build_only flag to run_tests.py instances.') |
| 405 | argp.add_argument('--force_default_poller', default=False, action='store_const', const=True, |
| 406 | help='Pass --force_default_poller to run_tests.py instances.') |
| 407 | argp.add_argument('--dry_run', |
| 408 | default=False, |
| 409 | action='store_const', |
| 410 | const=True, |
| 411 | help='Only print what would be run.') |
| 412 | argp.add_argument('--filter_pr_tests', |
| 413 | default=False, |
| 414 | action='store_const', |
| 415 | const=True, |
Jan Tattermusch | 68e27bf | 2016-12-16 14:09:03 +0100 | [diff] [blame] | 416 | help='Filters out tests irrelevant to pull request changes.') |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 417 | argp.add_argument('--base_branch', |
| 418 | default='origin/master', |
| 419 | type=str, |
| 420 | help='Branch that pull request is requesting to merge into') |
| 421 | argp.add_argument('--inner_jobs', |
| 422 | default=_DEFAULT_INNER_JOBS, |
| 423 | type=int, |
| 424 | help='Number of jobs in each run_tests.py instance') |
Jan Tattermusch | 68e27bf | 2016-12-16 14:09:03 +0100 | [diff] [blame] | 425 | argp.add_argument('-n', '--runs_per_test', default=1, type=_runs_per_test_type, |
| 426 | help='How many times to run each tests. >1 runs implies ' + |
| 427 | 'omitting passing test from the output & reports.') |
Craig Tiller | ceea969 | 2017-04-21 07:49:57 -0700 | [diff] [blame] | 428 | argp.add_argument('--max_time', default=-1, type=int, |
| 429 | help='Maximum amount of time to run tests for' + |
| 430 | '(other tests will be skipped)') |
Jan Tattermusch | ac9c9f9 | 2017-04-28 20:56:05 +0200 | [diff] [blame] | 431 | argp.add_argument('--internal_ci', |
| 432 | default=False, |
| 433 | action='store_const', |
| 434 | const=True, |
| 435 | help='Put reports into subdirectories to improve presentation of ' |
| 436 | 'results by Internal CI.') |
Matt Kwong | 52ff986 | 2017-04-17 13:56:51 -0700 | [diff] [blame] | 437 | argp.add_argument('--bq_result_table', |
| 438 | default='', |
| 439 | type=str, |
| 440 | nargs='?', |
| 441 | help='Upload test results to a specified BQ table.') |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 442 | args = argp.parse_args() |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 443 | |
Jan Tattermusch | ac9c9f9 | 2017-04-28 20:56:05 +0200 | [diff] [blame] | 444 | if args.internal_ci: |
| 445 | _report_filename = _report_filename_internal_ci # override the function |
| 446 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 447 | extra_args = [] |
| 448 | if args.build_only: |
| 449 | extra_args.append('--build_only') |
| 450 | if args.force_default_poller: |
| 451 | extra_args.append('--force_default_poller') |
Jan Tattermusch | 68e27bf | 2016-12-16 14:09:03 +0100 | [diff] [blame] | 452 | if args.runs_per_test > 1: |
| 453 | extra_args.append('-n') |
| 454 | extra_args.append('%s' % args.runs_per_test) |
| 455 | extra_args.append('--quiet_success') |
Craig Tiller | ceea969 | 2017-04-21 07:49:57 -0700 | [diff] [blame] | 456 | if args.max_time > 0: |
| 457 | extra_args.extend(('--max_time', '%d' % args.max_time)) |
Matt Kwong | 52ff986 | 2017-04-17 13:56:51 -0700 | [diff] [blame] | 458 | if args.bq_result_table: |
| 459 | extra_args.append('--bq_result_table') |
| 460 | extra_args.append('%s' % args.bq_result_table) |
| 461 | extra_args.append('--measure_cpu_costs') |
Matt Kwong | c5fd890 | 2017-08-10 13:39:21 -0700 | [diff] [blame] | 462 | extra_args.append('--disable_auto_set_flakes') |
Matt Kwong | fef9896 | 2016-10-27 10:45:47 -0700 | [diff] [blame] | 463 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 464 | all_jobs = _create_test_jobs(extra_args=extra_args, inner_jobs=args.inner_jobs) + \ |
| 465 | _create_portability_test_jobs(extra_args=extra_args, inner_jobs=args.inner_jobs) |
Jan Tattermusch | 6d7c6ef | 2016-09-22 13:40:48 +0200 | [diff] [blame] | 466 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 467 | jobs = [] |
| 468 | for job in all_jobs: |
| 469 | if not args.filter or all(filter in job.labels for filter in args.filter): |
Jan Tattermusch | 6a85129 | 2016-12-20 10:20:42 +0100 | [diff] [blame] | 470 | if not any(exclude_label in job.labels for exclude_label in args.exclude): |
| 471 | jobs.append(job) |
Jan Tattermusch | 6d7c6ef | 2016-09-22 13:40:48 +0200 | [diff] [blame] | 472 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 473 | if not jobs: |
| 474 | jobset.message('FAILED', 'No test suites match given criteria.', |
| 475 | do_newline=True) |
| 476 | sys.exit(1) |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 477 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 478 | print('IMPORTANT: The changes you are testing need to be locally committed') |
| 479 | print('because only the committed changes in the current branch will be') |
| 480 | print('copied to the docker environment or into subworkspaces.') |
murgatroid99 | 1687cab | 2016-10-11 11:42:01 -0700 | [diff] [blame] | 481 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 482 | skipped_jobs = [] |
Jan Tattermusch | 9c79e8d | 2016-09-19 14:33:18 +0200 | [diff] [blame] | 483 | |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 484 | if args.filter_pr_tests: |
| 485 | print('Looking for irrelevant tests to skip...') |
| 486 | relevant_jobs = filter_tests(jobs, args.base_branch) |
| 487 | if len(relevant_jobs) == len(jobs): |
| 488 | print('No tests will be skipped.') |
| 489 | else: |
| 490 | print('These tests will be skipped:') |
Matt Kwong | aa6c94c | 2016-11-09 15:53:23 -0800 | [diff] [blame] | 491 | skipped_jobs = list(set(jobs) - set(relevant_jobs)) |
| 492 | # Sort by shortnames to make printing of skipped tests consistent |
| 493 | skipped_jobs.sort(key=lambda job: job.shortname) |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 494 | for job in list(skipped_jobs): |
| 495 | print(' %s' % job.shortname) |
| 496 | jobs = relevant_jobs |
| 497 | |
| 498 | print('Will run these tests:') |
| 499 | for job in jobs: |
| 500 | if args.dry_run: |
| 501 | print(' %s: "%s"' % (job.shortname, ' '.join(job.cmdline))) |
| 502 | else: |
| 503 | print(' %s' % job.shortname) |
Matt Kwong | 5c691c6 | 2016-10-20 17:11:18 -0700 | [diff] [blame] | 504 | print |
| 505 | |
Jan Tattermusch | 7b9c21a | 2016-09-22 14:44:27 +0200 | [diff] [blame] | 506 | if args.dry_run: |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 507 | print('--dry_run was used, exiting') |
| 508 | sys.exit(1) |
| 509 | |
| 510 | jobset.message('START', 'Running test matrix.', do_newline=True) |
| 511 | num_failures, resultset = jobset.run(jobs, |
| 512 | newline_on_success=True, |
| 513 | travis=True, |
| 514 | maxjobs=args.jobs) |
| 515 | # Merge skipped tests into results to show skipped tests on report.xml |
| 516 | if skipped_jobs: |
Nathaniel Manista | 21e7dfc | 2017-03-12 18:24:09 +0000 | [diff] [blame] | 517 | ignored_num_skipped_failures, skipped_results = jobset.run( |
| 518 | skipped_jobs, skip_jobs=True) |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 519 | resultset.update(skipped_results) |
Jan Tattermusch | ac9c9f9 | 2017-04-28 20:56:05 +0200 | [diff] [blame] | 520 | report_utils.render_junit_xml_report(resultset, _report_filename('aggregate_tests'), |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 521 | suite_name='aggregate_tests') |
| 522 | |
| 523 | if num_failures == 0: |
| 524 | jobset.message('SUCCESS', 'All run_tests.py instance finished successfully.', |
| 525 | do_newline=True) |
Jan Tattermusch | 7b9c21a | 2016-09-22 14:44:27 +0200 | [diff] [blame] | 526 | else: |
Matt Kwong | 7e9bd6c | 2016-10-24 17:30:25 -0700 | [diff] [blame] | 527 | jobset.message('FAILED', 'Some run_tests.py instance have failed.', |
| 528 | do_newline=True) |
| 529 | sys.exit(1) |