openstack/election
Code Issues Proposed changes

Use the tag when grabbing the projects list

Browse Source 
The helper that added the tag to the cgit url was incorrectly appending
a simple "?tag" rather than "?h=tag"  cgit treats the tag as an unknown
option and is silently ignoring it.

Correct that.

The impact on the election is that any candidacies may have validated
may have included extra-atcs or repos added by the TV after the tagging
of the repo.  Checking the governance repo we see:

$ git log --no-merges --oneline aug-2017-elections..origin/master -- reference/projects.yaml
9b64bcb Adds oswin-tempest-plugin for Winstackers

This change clearly only affects the Winstackers candidates.

Change-Id: I1dfc47cb41aac4ce70d280e860ee4536cb90fd17
This commit is contained in:
Tony Breeds 2017-08-02 16:36:47 +10:00
parent b871ea7418
commit 21b862b005
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
openstack_election/utils.py
View File

@ -149,7 +149,7 @@ def get_projects(tag=None):
cache_file = '.projects.pkl'
if tag:
url += '?%s' % tag
url += '?h=%s' % tag
cache_file = '.projects.%s.pkl' % tag
# Refresh the cache if it's not there or if it's older than a week