From 32495f4ceecf4dbabaf86b20701a7301638da9f6 Mon Sep 17 00:00:00 2001 From: Jan Tattermusch Date: Tue, 23 Oct 2018 16:18:39 +0200 Subject: address review comments --- tools/interop_matrix/run_interop_matrix_tests.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'tools/interop_matrix') 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) -- cgit v1.2.3