Bug 1295536 - Update performance schema to treeherder latest;r=jmaher draft
authorWilliam Lachance <wlachance@mozilla.com>
Tue, 16 Aug 2016 13:30:06 -0400
changeset 404065 78ef5402e47e05f3f4e5a1db7b36e2d365c7b13b
parent 400825 054d4856cea6150a6638e5daf7913713281af97d
child 529082 cba96831e80944599ecf742cb9eec1913e882027
push id27097
push userwlachance@mozilla.com
push dateMon, 22 Aug 2016 19:31:18 +0000
reviewersjmaher
bugs1295536
milestone51.0a1
Bug 1295536 - Update performance schema to treeherder latest;r=jmaher This new version of the schema validates that numeric values are within an acceptable range (i.e. are not infinite) MozReview-Commit-ID: 32oT39Bfcwg
testing/mozharness/external_tools/performance-artifact-schema.json
--- a/testing/mozharness/external_tools/performance-artifact-schema.json
+++ b/testing/mozharness/external_tools/performance-artifact-schema.json
@@ -13,47 +13,57 @@
             "properties": {
                 "name": {
                     "title": "Subtest name",
                     "type": "string"
                 },
                 "value": {
                     "description": "Summary value for subtest",
                     "title": "Subtest value",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": -1000000000000.0,
+                    "maximum": 1000000000000.0
                 },
                 "lowerIsBetter": {
                     "description": "Whether lower values are better for subtest",
                     "title": "Lower is better",
                     "type": "boolean"
                 },
                 "shouldAlert": {
                     "description": "Whether we should alert",
                     "title": "Should alert",
                     "type": "boolean"
                 },
                 "alertThreshold": {
                     "description": "% change threshold before alerting",
                     "title": "Alert threshold",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": 0.0,
+                    "maximum": 1000.0
                 },
                 "minBackWindow": {
                     "description": "Minimum back window to use for alerting",
                     "title": "Minimum back window",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": 1,
+                    "maximum": 255
                 },
                 "maxBackWindow": {
                     "description": "Maximum back window to use for alerting",
                     "title": "Maximum back window",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": 1,
+                    "maximum": 255
                 },
                 "foreWindow": {
                     "description": "Fore window to use for alerting",
                     "title": "Fore window",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": 1,
+                    "maximum": 255
                 }
             },
             "required": [
                 "name",
                 "value"
             ],
             "type": "object"
         },
@@ -75,47 +85,57 @@
                     "items": {
                         "$ref": "#/definitions/subtest_schema"
                     },
                     "title": "Subtests",
                     "type": "array"
                 },
                 "value": {
                     "title": "Suite value",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": -1000000000000.0,
+                    "maximum": 1000000000000.0
                 },
                 "lowerIsBetter": {
                     "description": "Whether lower values are better for suite",
                     "title": "Lower is better",
                     "type": "boolean"
                 },
                 "shouldAlert": {
                     "description": "Whether we should alert on this suite (overrides default behaviour)",
                     "title": "Should alert",
                     "type": "boolean"
                 },
                 "alertThreshold": {
                     "description": "% change threshold before alerting",
                     "title": "Alert threshold",
-                    "type": "number"
+                    "type": "number",
+                    "minimum": 0.0,
+                    "maximum": 1000.0
                 },
                 "minBackWindow": {
                     "description": "Minimum back window to use for alerting",
                     "title": "Minimum back window",
-                    "type": "number"
+                    "type": "integer",
+                    "minimum": 1,
+                    "maximum": 255
                 },
                 "maxBackWindow": {
                     "description": "Maximum back window to use for alerting",
                     "title": "Maximum back window",
-                    "type": "number"
+                    "type": "integer",
+                    "minimum": 1,
+                    "maximum": 255
                 },
                 "foreWindow": {
                     "description": "Fore window to use for alerting",
                     "title": "Fore window",
-                    "type": "number"
+                    "type": "integer",
+                    "minimum": 1,
+                    "maximum": 255
                 }
             },
             "required": [
                 "name",
                 "subtests"
             ],
             "type": "object"
         }