From 210a0ea598d44d725518ee68a27ad185fb0c16b0 Mon Sep 17 00:00:00 2001
From: Jan Tattermusch <jtattermusch@google.com>
Date: Fri, 2 Oct 2015 15:05:36 -0700
Subject: [PATCH] tweaking of run_interop_tests.py

---
 tools/run_tests/dockerjob.py         |  5 ++++-
 tools/run_tests/run_interop_tests.py | 11 +++++++----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/tools/run_tests/dockerjob.py b/tools/run_tests/dockerjob.py
index e938a3c2f7f..11686d46b09 100755
--- a/tools/run_tests/dockerjob.py
+++ b/tools/run_tests/dockerjob.py
@@ -44,7 +44,10 @@ def wait_for_file(filepath, timeout_seconds=15):
   while time.time() - started < timeout_seconds:
     if os.path.isfile(filepath):
       with open(filepath, 'r') as f:
-        return f.read()
+        content = f.read()
+        # make sure we don't return empty content
+        if content:
+          return content
     time.sleep(1)
   raise Exception('Failed to read file %s.' % filepath)
 
diff --git a/tools/run_tests/run_interop_tests.py b/tools/run_tests/run_interop_tests.py
index 274eaa33363..46b34fea4c0 100755
--- a/tools/run_tests/run_interop_tests.py
+++ b/tools/run_tests/run_interop_tests.py
@@ -35,6 +35,7 @@ import dockerjob
 import itertools
 import xml.etree.cElementTree as ET
 import jobset
+import multiprocessing
 import os
 import subprocess
 import sys
@@ -224,10 +225,11 @@ _LANGUAGES = {
 # TODO(jtattermusch): enable other languages as servers as well
 _SERVERS = ['c++', 'node', 'csharp', 'java']
 
+# TODO(jtattermusch): add empty_stream once PHP starts supporting it.
 # TODO(jtattermusch): add timeout_on_sleeping_server once java starts supporting it.
 # TODO(jtattermusch): add support for auth tests.
 _TEST_CASES = ['large_unary', 'empty_unary', 'ping_pong',
-               'empty_stream', 'client_streaming', 'server_streaming',
+               'client_streaming', 'server_streaming',
                'cancel_after_begin', 'cancel_after_first_response']
 
 
@@ -274,7 +276,7 @@ def cloud_to_prod_jobspec(language, test_case, docker_image=None):
           cwd=cwd,
           environ=environ,
           shortname="cloud_to_prod:%s:%s" % (language, test_case),
-          timeout_seconds=60,
+          timeout_seconds=2*60,
           flake_retries=5 if args.allow_flakes else 0,
           timeout_retries=2 if args.allow_flakes else 0)
   return test_job
@@ -299,7 +301,7 @@ def cloud_to_cloud_jobspec(language, test_case, server_name, server_host,
           cwd=cwd,
           shortname="cloud_to_cloud:%s:%s_server:%s" % (language, server_name,
                                                  test_case),
-          timeout_seconds=60,
+          timeout_seconds=2*60,
           flake_retries=5 if args.allow_flakes else 0,
           timeout_retries=2 if args.allow_flakes else 0)
   return test_job
@@ -346,7 +348,7 @@ argp.add_argument('-l', '--language',
                   nargs='+',
                   default=['all'],
                   help='Clients to run.')
-argp.add_argument('-j', '--jobs', default=24, type=int)
+argp.add_argument('-j', '--jobs', default=multiprocessing.cpu_count(), type=int)
 argp.add_argument('--cloud_to_prod',
                   default=False,
                   action='store_const',
@@ -437,6 +439,7 @@ try:
     server_jobs[lang] = job
     server_addresses[lang] = ('localhost', job.mapped_port(_DEFAULT_SERVER_PORT))
 
+
   jobs = []
   if args.cloud_to_prod:
     for language in languages: