From 1198da8eefc597b135ed7c59c8ec3d568d3197fd Mon Sep 17 00:00:00 2001 From: Jimmy Berry Date: Tue, 25 Apr 2017 19:15:04 -0500 Subject: [PATCH] cleanup_ring: extract repo_state_acceptable(). --- osclib/cleanup_rings.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/osclib/cleanup_rings.py b/osclib/cleanup_rings.py index c18beaf2..ee6de2bd 100644 --- a/osclib/cleanup_rings.py +++ b/osclib/cleanup_rings.py @@ -88,8 +88,8 @@ class CleanupRings(object): b = self.bin2src[pkg.text] self.pkgdeps[b] = source - def check_depinfo_ring(self, prj, nextprj): - url = makeurl(self.api.apiurl, ['build', prj, '_result']) + def repo_state_acceptable(self, project): + url = makeurl(self.api.apiurl, ['build', project, '_result']) root = ET.parse(http_GET(url)).getroot() for repo in root.findall('result'): repostate = repo.get('state', 'missing') @@ -101,6 +101,11 @@ class CleanupRings(object): if code not in ['succeeded', 'excluded', 'disabled']: print('Package {}/{}/{} is {}'.format(repo.get('project'), repo.get('repository'), package.get('package'), code)) return False + return True + + def check_depinfo_ring(self, prj, nextprj): + if not self.repo_state_acceptable(prj): + return False self.find_inner_ring_links(prj) for arch in self.api.cstaging_dvd_archs: