aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar David Garcia Quintas <dgq@google.com>2017-08-16 12:10:48 -0700
committerGravatar David Garcia Quintas <dgq@google.com>2017-08-16 12:10:48 -0700
commitbfd9d805a16266b34adbc1f51eaf59d523276027 (patch)
treefd871ce4a87cc947b725519115a98539fd5dc27e
parent4721fc07169c608606cabdd905eaf69fb8cd72ab (diff)
Fix handling of no new flakes
-rw-r--r--tools/flakes/detect_flakes.py13
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__':