opendev
/
jeepyb
Code Issues Proposed changes

Hound config: disambiguate key names for projects in new namespaces 

Because we use os.path.basename() for the key in the config
dictionary, openstack/project-config and zuul/project-config map to
the same key and overwrite each other, thus we only get indexing on
one or the other.

Use the full project name as the key instead.

Change-Id: I8a32ddc2e24211cc9300a2f90df19e1354251ae5
This commit is contained in:
Ian Wienand 2019-05-13 09:30:09 +10:00
parent 1d08f4e920
commit 12bd0e0f35
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
jeepyb/cmd/create_hound_config.py
View File

@ -37,13 +37,12 @@ def main():
# active anymore.
if project.startswith(('openstack-attic', 'stackforge')):
continue
basename = os.path.basename(project)
# ignore deb- projects that are forks of other projects intended for
# internal debian packaging needs only and are generally not of
# interest to upstream developers
if basename.startswith('deb-'):
if os.path.basename(project).startswith('deb-'):
continue
repos[basename] = {
repos[project] = {
'url': "%(proto)s%(gitbase)s/%(project)s" % dict(
proto=GIT_PROTOCOL, gitbase=GIT_SERVER, project=project),
'url-pattern': {