From 851c992d06a15f9c84482f6c926d8cce72e63de1 Mon Sep 17 00:00:00 2001 From: Qian Sun Date: Fri, 3 Feb 2017 11:40:42 -0800 Subject: [PATCH] Modify test justification. Change-Id: If0155270446c80756ce33910fb27efa3e6f3852f --- test/client/esp_client.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/test/client/esp_client.py b/test/client/esp_client.py index 4c6bfeb2d..35efda904 100755 --- a/test/client/esp_client.py +++ b/test/client/esp_client.py @@ -118,10 +118,12 @@ def count_failed_requests(out): """ Count failed and non-2xx responses """ failed = 0 non2xx = 0 + completed = 0 for metrics, _, _ in out: failed += metrics.get('Failed requests', [0])[0] non2xx += metrics.get('Non-2xx responses', [0])[0] - return failed, non2xx + completed += metrics.get('Complete requests', [0])[0] + return failed, non2xx, completed if __name__ == "__main__": try: @@ -186,8 +188,8 @@ def count_failed_requests(out): if FLAGS.test_env: esp_perfkit_publisher.Publish(results, test_env) - failed, non2xx = count_failed_requests(results) - if failed + non2xx > 0: + failed, non2xx, completed = count_failed_requests(results) + if failed + non2xx > 0.005 * completed: sys.exit( ('Load test failed:\n' ' {} failed requests,\n'