Merge ~paelzer/ubuntu-helpers:fix-unparsable-jobs into ~ubuntu-server/+git/ubuntu-helpers:main

[Christian Ehrhardt ]

[Bryce Harrington (bryce)]

Is the broken json something that occurs often? I think catching that is fine in this case but if it happens a lot might be something worth investigating at some point.

The private jobs I've run into too, and the handling looks good.

review: Approve

[Christian Ehrhardt  (paelzer)]

I've seen it somewhere in the <0.5% range. Maybe even way less and it was a one off thing on the backend.

[Christian Ehrhardt  (paelzer)]

Merged

diff --git a/cpaelzer/lp-test-isrunning b/cpaelzer/lp-test-isrunning
index abc260b..4b43221 100755
--- a/cpaelzer/lp-test-isrunning
+++ b/cpaelzer/lp-test-isrunning
@@ -60,11 +60,17 @@ with urllib.request.urlopen(request) as response:
             for arch in queues[origin][series]:
                 n = 0
                 for key in queues[origin][series][arch]:
-                    (pkg, json_data) = key.split(maxsplit=1)
-                    jobinfo = json.loads(json_data)
-
-                    triggers = ','.join(jobinfo.get('triggers', '-'))
-                    ppas = ','.join(jobinfo.get('ppas', '-'))
+                    if key == "private job":
+                        pkg = triggers = ppas = "private job"
+                    else:
+                        (pkg, json_data) = key.split(maxsplit=1)
+                        try:
+                            jobinfo = json.loads(json_data)
+                            triggers = ','.join(jobinfo.get('triggers', '-'))
+                            ppas = ','.join(jobinfo.get('ppas', '-'))
+                        except json.decoder.JSONDecodeError:
+                            pkg = triggers = ppas = "failed to parse"
+                            continue
 
                     n = n + 1
                     try: