openstack/releases
Code Issues Proposed changes

Merge "only fail pypi permission validation if the project exists"

Browse Source
This commit is contained in:
Zuul 2019-03-08 00:31:58 +00:00 committed by Gerrit Code Review
commit f4ccd33649
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
openstack_releases/cmds/validate.py
View File

@ -860,10 +860,16 @@ def validate_pypi_permissions(deliv, context):
uploaders = pythonutils.get_pypi_uploaders(pypi_name) uploaders = pythonutils.get_pypi_uploaders(pypi_name)
if not uploaders: if not uploaders:
context.error( pypi_info = pythonutils.get_pypi_info(pypi_name)
'could not find users with permission to upload packages ' if not pypi_info:
'for {}. Is the sdist name correct?'.format(pypi_name) LOG.debug('no %s project data on pypi, assuming it will be '
) 'created by release',
pypi_name)
else:
context.error(
'could not find users with permission to upload packages '
'for {}. Is the sdist name correct?'.format(pypi_name)
)
elif 'openstackci' not in uploaders: elif 'openstackci' not in uploaders:
context.error( context.error(
'openstackci does not have permission to upload packages ' 'openstackci does not have permission to upload packages '