Domain Reliability: Reenable AddBakedInConfigs test

components/domain_reliability/bake_in_configs.py

Index: components/domain_reliability/bake_in_configs.py
diff --git a/components/domain_reliability/bake_in_configs.py b/components/domain_reliability/bake_in_configs.py
index 9990d88dbd4ff2c91c905401eaef3d64dec8cf50..291e1ae11c9c25711988e22fb701830ff3e9a4ec 100755
--- a/components/domain_reliability/bake_in_configs.py
+++ b/components/domain_reliability/bake_in_configs.py
@@ -533,9 +533,10 @@ def main():
       json_text = f.read()
     try:
       config = json.loads(json_text)
-    except ValueError:
-      print >> sys.stderr, "%s: error parsing JSON" % json_file
-      raise
+    except ValueError, e:
+      print >> sys.stderr, "%s: error parsing JSON: %s" % (json_file, e)
+      found_invalid_config = True
+      continue

[davidben, 2014/10/20 20:51:31]

Nit: could you add a sentence about this in the CL description? It doesn't seem
related to re-enabling it. (I suppose it could also be split out, but whatever.)

[Deprecated (see juliatuttle), 2014/10/20 21:07:47]

Done.

     if 'monitored_domain' not in config:
       print >> sys.stderr, '%s: no monitored_domain found' % json_file
       found_invalid_config = True