modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-04-02 10:20:31 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix an error in the KojiBuilder.

Browse Source 
The issue is that _get_task_by_artifact returns either None or a dict of
info and we weren't handling that correctly.  My mistake.
This commit is contained in:
Ralph Bean
2016-11-02 10:23:42 -04:00
parent b2d1cf9f0d
commit 28183b9d3f
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
module_build_service/builder.py
View File

@@ -579,11 +579,10 @@ chmod 644 %buildroot/%_rpmconfigdir/macros.d/macros.modules
# Skip existing builds
task_info = self._get_task_by_artifact(artifact_name)
task_id = task_info['task_id']
if task_id:
if task_info:
log.info("skipping build of %s. Build already exists (task_id=%s), via %s" % (
source, task_id, self))
return task_id, koji.BUILD_STATES['COMPLETE'], 'Build already exists.', task_info['nvr']
source, task_info['task_id'], self))
return task_info['task_id'], koji.BUILD_STATES['COMPLETE'], 'Build already exists.', task_info['nvr']
self._koji_whitelist_packages([artifact_name,])
if '://' not in source: