aboutsummaryrefslogtreecommitdiffhomepage
path: root/tools/run_tests/python_utils
diff options
context:
space:
mode:
authorGravatar Matt Kwong <matt-kwong@users.noreply.github.com>2018-05-14 10:46:53 -0700
committerGravatar GitHub <noreply@github.com>2018-05-14 10:46:53 -0700
commit77461d45979cab23cc1a7954a147a4d3ac25fdaf (patch)
tree01b85b7fa6b0e0a386e2c7965333251628381bb1 /tools/run_tests/python_utils
parentdcd5aefba3dcce196613f1b909b4458413b88808 (diff)
parent5aafb3b61c22fc0ff7edacd008beda775d631d97 (diff)
Merge pull request #15345 from matt-kwong/rbe-upload
Handle timeouts when uploading ResultStore results
Diffstat (limited to 'tools/run_tests/python_utils')
-rw-r--r--tools/run_tests/python_utils/upload_rbe_results.py12
1 files changed, 11 insertions, 1 deletions
diff --git a/tools/run_tests/python_utils/upload_rbe_results.py b/tools/run_tests/python_utils/upload_rbe_results.py
index d302024883..5955b3792f 100644
--- a/tools/run_tests/python_utils/upload_rbe_results.py
+++ b/tools/run_tests/python_utils/upload_rbe_results.py
@@ -146,7 +146,15 @@ if __name__ == "__main__":
test_cases = [{
'testCase': {
'caseName': str(action['id']['actionId']),
- 'result': str(action['statusAttributes']['status'])
+ }
+ }]
+ # Test timeouts have a different dictionary structure compared to pass and
+ # fail results.
+ elif action['statusAttributes']['status'] == 'TIMED_OUT':
+ test_cases = [{
+ 'testCase': {
+ 'caseName': str(action['id']['actionId']),
+ 'timedOut': True
}
}]
else:
@@ -155,6 +163,8 @@ if __name__ == "__main__":
for test_case in test_cases:
if 'errors' in test_case['testCase']:
result = 'FAILED'
+ elif 'timedOut' in test_case['testCase']:
+ result = 'TIMEOUT'
else:
result = 'PASSED'
bq_rows.append({