From 8bcbee815c5d7aa6cda13371241536db6dc03fbb Mon Sep 17 00:00:00 2001
From: Sree Kuchibhotla <sreek@google.com>
Date: Wed, 24 Feb 2016 15:52:54 -0800
Subject: [PATCH] Fix a bug in failure check

---
 tools/gke/run_stress_tests_on_gke.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/gke/run_stress_tests_on_gke.py b/tools/gke/run_stress_tests_on_gke.py
index 0ea7b7fcc1..9a8a33cac5 100755
--- a/tools/gke/run_stress_tests_on_gke.py
+++ b/tools/gke/run_stress_tests_on_gke.py
@@ -244,7 +244,7 @@ def _launch_client(gcp_project_id, docker_image_name, bq_settings,
   client_arg_list = []
 
   # TODO(sree) Make this configurable (and also less frequent)
-  poll_interval_secs = 5
+  poll_interval_secs = 30
 
   metrics_port = 8081
   metrics_server_address = 'localhost:%d' % metrics_port
@@ -397,7 +397,7 @@ def run_test(skip_building_image, gcp_project_id, image_name, tag_name,
       break
 
     # Check if either stress server or clients have failed
-    if not bq_helper.check_if_any_tests_failed():
+    if bq_helper.check_if_any_tests_failed():
       is_success = False
       print 'Some tests failed.'
       break
-- 
GitLab