diff options
author | 2017-08-16 12:10:48 -0700 | |
---|---|---|
committer | 2017-08-16 12:10:48 -0700 | |
commit | bfd9d805a16266b34adbc1f51eaf59d523276027 (patch) | |
tree | fd871ce4a87cc947b725519115a98539fd5dc27e | |
parent | 4721fc07169c608606cabdd905eaf69fb8cd72ab (diff) |
Fix handling of no new flakes
-rw-r--r-- | tools/flakes/detect_flakes.py | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/tools/flakes/detect_flakes.py b/tools/flakes/detect_flakes.py index 32899074da..2aff4c0872 100644 --- a/tools/flakes/detect_flakes.py +++ b/tools/flakes/detect_flakes.py @@ -68,10 +68,13 @@ ORDER BY timestamp desc query_job = big_query_utils.sync_query_job(bq, 'grpc-testing', query) page = bq.jobs().getQueryResults( pageToken=None, **query_job['jobReference']).execute(num_retries=3) - testname_to_cols = {row['f'][0]['v']: - (row['f'][1]['v'], row['f'][2]['v'], row['f'][3]['v']) - for row in page['rows']} - return testname_to_cols + rows = page.get('rows') + if rows: + return {row['f'][0]['v']: + (row['f'][1]['v'], row['f'][2]['v'], row['f'][3]['v']) + for row in rows} + else: + return {} def get_new_flakes(): @@ -91,6 +94,8 @@ def main(): if new_flakes: print("Found {} new flakes:".format(len(new_flakes))) print_table(new_flakes) + else: + print("No new flakes found!") if __name__ == '__main__': |