yudaike/openSUSE-release-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #893 from jberry-suse/followup-devel-non-bootstrapped

Browse Source 
request_splitter: only allow devel/super followup for non-bootstrapped.
This commit is contained in:
Ludwig Nussel 2017-05-10 11:15:50 +02:00 committed by GitHub
commit 9c3c04b7c1
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
osclib/request_splitter.py
View File

@ -176,8 +176,9 @@ class RequestSplitter(object):
def is_staging_mergeable(self, status, pseudometa):
return len(pseudometa['requests']) > 0 and 'splitter_info' in pseudometa
def should_staging_merge(self, status, pseudometa):
if (pseudometa['splitter_info']['strategy']['name'] in ('devel', 'super') and
def should_staging_merge(self, status, pseudometa, bootstrapped):
if (not bootstrapped and
pseudometa['splitter_info']['strategy']['name'] in ('devel', 'super') and
status['overall_state'] not in ('acceptable', 'review')):
# Simplistic attempt to allow for followup requests to be staged
# after age max has been passed while still stopping when ready.
@ -225,18 +226,19 @@ class RequestSplitter(object):
for staging in stagings:
project = self.api.prj_from_short(staging)
status, pseudometa = self.staging_status_load(project)
bootstrapped = self.is_staging_bootstrapped(project)
# Store information about staging.
self.stagings[staging] = {
'project': project,
'bootstrapped': self.is_staging_bootstrapped(project),
'bootstrapped': bootstrapped,
'status': status,
'pseudometa': pseudometa,
}
# Decide if staging of interested.
if self.is_staging_mergeable(status, pseudometa) and (
should_always or self.should_staging_merge(status, pseudometa)):
should_always or self.should_staging_merge(status, pseudometa, bootstrapped)):
if pseudometa['splitter_info']['strategy']['name'] == 'none':
self.stagings_mergeable_none.append(staging)
else: