Merge ~fginther/+git/ksc-test-results:fginther/allow-missing-bug-target into ~canonical-kernel/+git/ksc-test-results:main

[Francis Ginther]

sru_cycle_status: Don't crash on bugs missing a target field

When calculating the active sru_cycles, we sometimes encounter a bug
with a missing `target` field. Log the bug, but don't crash.

Signed-off-by: Francis Ginther <email address hidden>

https://warthogs.atlassian.net/browse/KERNTT-255

[Cory Todd (corytodd)]

LGTM. It will be interesting to see if there is any pattern to the offending requests.

review: Approve

diff --git a/stable-reports/lib/sru_cycle_status.py b/stable-reports/lib/sru_cycle_status.py
index 11baa0b..cd98e15 100644
--- a/stable-reports/lib/sru_cycle_status.py
+++ b/stable-reports/lib/sru_cycle_status.py
@@ -50,10 +50,17 @@ class SRUCycleStatus:
             #
             # Check that variant is present in the bug. This should not
             # be possible but some bugs hit this state.
-            if "variant" in bug and bug["variant"] == "snap-debs":
-                cycles[cycle][series]["order"][f"{bug['package']} / {bug['target']}"] = id
-            else:
-                cycles[cycle][series]["order"][bug["package"]] = id
+            try:
+                if "variant" in bug and bug["variant"] == "snap-debs":
+                    cycles[cycle][series]["order"][f"{bug['package']} / {bug['target']}"] = id
+                else:
+                    cycles[cycle][series]["order"][bug["package"]] = id
+            except KeyError as e:
+                # Found a bug in a transient or broken state with a missing target
+                # This is not critical to populating the cycle list
+                with open("/tmp/stable-report-missing-target-bugs.txt", "w") as bug_file:
+                    bug_file.write("bug: {} `{}`\n".format(id, str(e)))
+                continue
 
             # --- reasons ---