diff options
author | Jan Tattermusch <jtattermusch@google.com> | 2018-10-23 16:18:39 +0200 |
---|---|---|
committer | Jan Tattermusch <jtattermusch@google.com> | 2018-10-23 16:18:39 +0200 |
commit | 32495f4ceecf4dbabaf86b20701a7301638da9f6 (patch) | |
tree | 9c0621023935a56cbd8addc4efee2ebd1d6cdba9 /tools/interop_matrix | |
parent | 6b5f1143fc6d7ac95ec244e71b0995a4fcb878f1 (diff) |
address review comments
Diffstat (limited to 'tools/interop_matrix')
-rwxr-xr-x | tools/interop_matrix/run_interop_matrix_tests.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tools/interop_matrix/run_interop_matrix_tests.py b/tools/interop_matrix/run_interop_matrix_tests.py index d5388a22f7..be9edc0b9b 100755 --- a/tools/interop_matrix/run_interop_matrix_tests.py +++ b/tools/interop_matrix/run_interop_matrix_tests.py @@ -165,6 +165,8 @@ def _generate_test_case_jobspecs(lang, runtime, release, suite_name): job_spec_list = [] for line in testcase_lines: + # TODO(jtattermusch): revisit the logic for updating test case commands + # what it currently being done seems fragile. m = re.search('--test_case=(.*)"', line) shortname = m.group(1) if m else 'unknown_test' m = re.search('--server_host_override=(.*).sandbox.googleapis.com', @@ -198,8 +200,8 @@ def _pull_images_for_lang(lang, images): download_specs = [] for release, image in images: # Pull the image and warm it up. - # First time we use an image with "docker run", it takes time to unpack the image - # and later this delay would fail our test cases. + # First time we use an image with "docker run", it takes time to unpack + # the image and later this delay would fail our test cases. cmdline = [ 'gcloud docker -- pull %s && docker run --rm=true %s /bin/true' % (image, image) |