From 00f6bdd35865d065850633cf37863aacac182d6f Mon Sep 17 00:00:00 2001 From: Lenny Verkhovsky Date: Thu, 17 Dec 2015 17:46:30 +0200 Subject: [PATCH] Fixed quering new Gerrit Extend rest query with additional options to get 'name' that was removed with commit https://gerrit-review.googlesource.com/#/c/39767/ Removed 'exit' on invalid gerrit reviewer name. Change-Id: If48d5655202ca1e36c33ac6067e86b2db84360f2 --- monitoring/lastcomment-scoreboard/lastcomment.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/monitoring/lastcomment-scoreboard/lastcomment.py b/monitoring/lastcomment-scoreboard/lastcomment.py index 0029044..40c0129 100755 --- a/monitoring/lastcomment-scoreboard/lastcomment.py +++ b/monitoring/lastcomment-scoreboard/lastcomment.py @@ -72,13 +72,13 @@ def query_gerrit(name, count, project): if project: search = search + (" AND project:\"%s\"" % project) query = ("https://review.openstack.org/changes/?q=%s&" - "o=MESSAGES" % search) + "o=MESSAGES&o=DETAILED_ACCOUNTS" % search) r = requests.get(query) try: changes = json.loads(r.text[4:]) except ValueError: print "query: '%s' failed with:\n%s" % (query, r.text) - sys.exit(1) + return [] comments = [] for change in changes: