tyler.durden/compose
1
0
Fork 0
mirror of https://github.com/docker/compose.git synced 2025-07-27 07:34:10 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix some release script issues

Browse Source 
Signed-off-by: Joffrey F <joffrey@docker.com>
This commit is contained in:
Joffrey F 2018-09-24 20:05:40 -07:00
parent ec4ea8d2f1
commit 47d740b800
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
script/release/release.py
View File

@ -77,19 +77,24 @@ def monitor_pr_status(pr_data):
'pending': 0, 'pending': 0,
'success': 0, 'success': 0,
'failure': 0, 'failure': 0,
'error': 0,
} }
for detail in status.statuses: for detail in status.statuses:
if detail.context == 'dco-signed': if detail.context == 'dco-signed':
# dco-signed check breaks on merge remote-tracking ; ignore it # dco-signed check breaks on merge remote-tracking ; ignore it
continue continue
summary[detail.state] += 1 if detail.state in summary:
print('{pending} pending, {success} successes, {failure} failures'.format(**summary)) summary[detail.state] += 1
if summary['pending'] == 0 and summary['failure'] == 0 and summary['success'] > 0: print(
'{pending} pending, {success} successes, {failure} failures, '
'{error} errors'.format(**summary)
)
if summary['failure'] > 0 or summary['error'] > 0:
raise ScriptError('CI failures detected!')
elif summary['pending'] == 0 and summary['success'] > 0:
# This check assumes at least 1 non-DCO CI check to avoid race conditions. # This check assumes at least 1 non-DCO CI check to avoid race conditions.
# If testing on a repo without CI, use --skip-ci-check to avoid looping eternally # If testing on a repo without CI, use --skip-ci-check to avoid looping eternally
return True return True
elif summary['failure'] > 0:
raise ScriptError('CI failures detected!')
time.sleep(30) time.sleep(30)
elif status.state == 'success': elif status.state == 'success':
print('{} successes: all clear!'.format(status.total_count)) print('{} successes: all clear!'.format(status.total_count))
@ -97,12 +102,14 @@ def monitor_pr_status(pr_data):
def check_pr_mergeable(pr_data): def check_pr_mergeable(pr_data):
if not pr_data.mergeable: if pr_data.mergeable is False:
# mergeable can also be null, in which case the warning would be a false positive.
print( print(
'WARNING!! PR #{} can not currently be merged. You will need to ' 'WARNING!! PR #{} can not currently be merged. You will need to '
'resolve the conflicts manually before finalizing the release.'.format(pr_data.number) 'resolve the conflicts manually before finalizing the release.'.format(pr_data.number)
) )
return pr_data.mergeable
return pr_data.mergeable is True
def create_release_draft(repository, version, pr_data, files): def create_release_draft(repository, version, pr_data, files):