Bug 1313426 - gecko decision task: ValueError: No JSON object. fix SETA. r=dustin on a CLOSED TREE
authorJoel Maher <jmaher@mozilla.com>
Thu, 27 Oct 2016 15:12:33 -0400
changeset 319877 a8c8bb56737d5f5407d80c731bfe70fa3cba7f77
parent 319875 582e5f96a727af4a6da4ec26dbe882ef57914bfb
child 319878 da1c0184e9f51860b8ac0b75c78bb30ddeed473c
push id20749
push userryanvm@gmail.com
push dateSat, 29 Oct 2016 13:21:21 +0000
treeherderfx-team@1b170b39ed6b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdustin
bugs1313426
milestone52.0a1
Bug 1313426 - gecko decision task: ValueError: No JSON object. fix SETA. r=dustin on a CLOSED TREE MozReview-Commit-ID: F4V3VcDHMnW
taskcluster/taskgraph/util/seta.py
--- a/taskcluster/taskgraph/util/seta.py
+++ b/taskcluster/taskgraph/util/seta.py
@@ -59,16 +59,20 @@ class SETA(object):
         except exceptions.HTTPError:
             logger.warning("We got bad Http response from ouija,"
                            " we will treat all test tasks as high value.")
 
         # We just print the error out as a debug message if we failed to catch the exception above
         except exceptions.RequestException as error:
             logger.warning(error)
 
+        # When we get invalid JSON (i.e. 500 error), it results in a ValueError (bug 1313426)
+        except ValueError as error:
+            logger.warning("Invalid JSON, possible server error: {}".format(error))
+
         return low_value_tasks
 
     def is_low_value_task(self, label, project):
         # cache the low value tasks per project to avoid repeated SETA server queries
         if project not in self.low_value_tasks:
             self.low_value_tasks[project] = self.query_low_value_tasks(project)
         return label in self.low_value_tasks[project]