From c39f1d6d59c9142adadf30ca34f993c645ab9859 Mon Sep 17 00:00:00 2001 From: Christian Kellner Date: Sun, 13 Sep 2020 18:40:13 +0200 Subject: [PATCH] plugin/builder: properly raise exceptions In two cases the exception was merely instantiated but not actually thrown. --- plugins/builder/osbuild.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/builder/osbuild.py b/plugins/builder/osbuild.py index 286885e..ba9d10d 100644 --- a/plugins/builder/osbuild.py +++ b/plugins/builder/osbuild.py @@ -257,7 +257,7 @@ class OSBuildImage(BaseTaskHandler): target_info = self.session.getBuildTarget(target, strict=True) if not target_info: - koji.BuildError(f"Target '{target}' not found") + raise koji.BuildError(f"Target '{target}' not found") build_tag = target_info['build_tag'] buildconfig = self.session.getBuildConfig(build_tag) @@ -267,7 +267,7 @@ class OSBuildImage(BaseTaskHandler): arches = set(arches) diff = tag_arches - arches if diff: - koji.BuildError("Unsupported architecture(s): " + str(diff)) + raise koji.BuildError("Unsupported architecture(s): " + str(diff)) # Repositories repo_urls = opts.get("repo")