From bc2be18261fbbe7815a84fdac3c69a7b5c03149f Mon Sep 17 00:00:00 2001
From: Jeremy Cline <jeremycline@linux.microsoft.com>
Date: Aug 29 2024 15:46:02 +0000
Subject: Skip composes where we can't get the release info via compose ID


Production is currently boot-looping since fedfind is raising a
ValueError with the message "Cannot find ELN composes by compose ID!".
For now, just skip these messages. In the future we will have to figure
out how to handle them more gracefully if we want to upload ELN images.

---

diff --git a/fedora-image-uploader/fedora_image_uploader/handler.py b/fedora-image-uploader/fedora_image_uploader/handler.py
index 6a1b311..9b8d9af 100644
--- a/fedora-image-uploader/fedora_image_uploader/handler.py
+++ b/fedora-image-uploader/fedora_image_uploader/handler.py
@@ -117,6 +117,12 @@ class Uploader:
         except ff_exceptions.UnsupportedComposeError:
             _log.info("Skipping compose %s as it contains no images", compose_id)
             return
+        except ValueError as e:
+            # fedfind raises ValueError if, for example, this is an ELN compose
+            # (it cannot find ELN composes by compose ID). Probably need to handle
+            # this differently when we add support for ELN images.
+            _log.warning("Failed to retrieve the release via compose-id: %s", str(e))
+            return
         # reset for each message
         self.container_repos = dict()
         try: