From 4747d3ee4317f3748f9ef0e20068b4ab5b7dd485 Mon Sep 17 00:00:00 2001 From: synkd Date: Wed, 6 Sep 2023 12:04:17 -0400 Subject: [PATCH] Modify content and return value after timeout This PR expicitly sets manifest.content to `None` and returns the manifest object when a manifest export job times out. This change facilitates the failover to cloned manifests in Robottelo. Previously, failed export jobs triggered an exception in Robottelo when manifester fixtures attempted to upload manifests because, in those cases, the manifest object did not contain a `content` attribute. --- manifester/manifester.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/manifester/manifester.py b/manifester/manifester.py index 04bd002..5cc3747 100644 --- a/manifester/manifester.py +++ b/manifester/manifester.py @@ -349,7 +349,8 @@ def trigger_manifest_export(self): break request_count += 1 if limit_exceeded: - return + self.content = None + return self export_job = export_job.json() export_href = export_job["body"]["href"] manifest = simple_retry(