From d263d0ce455d9a7372301fa0bd3eedc5e0f80221 Mon Sep 17 00:00:00 2001 From: Jeremy Stanley Date: Wed, 1 Nov 2017 18:35:00 +0000 Subject: [PATCH] Use cgit more in contributor listing The change which just merged to incorporate recent retired deliverables and removed projects in active contributor analysis was written prior to the gitweb->cgit transition, so fix up that feature to also use the new cgit-based function call. Change-Id: Ifde87147399967c43f3e4c93de4347acd31e73cc --- tools/owners.py | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/tools/owners.py b/tools/owners.py index b4e33c3569..50e0a194fa 100644 --- a/tools/owners.py +++ b/tools/owners.py @@ -340,13 +340,9 @@ def main(argv=sys.argv): # are merged into the main dict if their retired-on date falls # later than the after parameter for the qualifying time period # TODO(fungi): make this a configurable option - old_projects = query_gerrit( - 'gitweb', { - 'p': 'openstack/governance.git', - 'a': 'blob_plain', - 'f': 'reference/legacy.yaml', - 'hb': ref, - }) + old_projects = get_from_cgit('openstack/governance', + 'reference/legacy.yaml', + {'h': ref}) for project in old_projects: for deliverable in old_projects[project]['deliverables']: if 'retired-on' in old_projects[project]['deliverables'][deliverable]: