From a1ebd234a4f9e64f4476d827e5c450a7c02157c8 Mon Sep 17 00:00:00 2001 From: Lubomír Sedlář Date: Sep 10 2021 09:29:53 +0000 Subject: Only build CTS url when configured JIRA: RHELCMP-6611 Signed-off-by: Lubomír Sedlář --- diff --git a/pungi/compose.py b/pungi/compose.py index a43d899..37f6344 100644 --- a/pungi/compose.py +++ b/pungi/compose.py @@ -157,14 +157,15 @@ def update_compose_url(compose_dir, conf): compose_id = f.read() authentication = get_authentication(conf) cts_url = conf.get("cts_url", None) - url = os.path.join(cts_url, "api/1/composes", compose_id) - tp = conf.get("translate_paths", None) - compose_url = translate_path_raw(tp, compose_dir) - data = { - "action": "set_url", - "compose_url": compose_url, - } - return requests.patch(url, json=data, auth=authentication) + if cts_url: + url = os.path.join(cts_url, "api/1/composes", compose_id) + tp = conf.get("translate_paths", None) + compose_url = translate_path_raw(tp, compose_dir) + data = { + "action": "set_url", + "compose_url": compose_url, + } + return requests.patch(url, json=data, auth=authentication) def get_compose_dir( diff --git a/pungi/scripts/pungi_koji.py b/pungi/scripts/pungi_koji.py index b8d899b..6f1b92a 100644 --- a/pungi/scripts/pungi_koji.py +++ b/pungi/scripts/pungi_koji.py @@ -330,7 +330,7 @@ def main(): ) rv = Compose.update_compose_url(compose_dir, conf) - if not rv.ok: + if rv and not rv.ok: logger.error("CTS compose_url update failed with the error: %s" % rv.text) errors, warnings = pungi.checks.validate(conf, offline=False)