Teach Buildbot dashboard to parse alternative revision format
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jan 2014 19:06:40 +0000 (19:06 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Jan 2014 19:06:40 +0000 (19:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=127561

Reviewed by Alexey Proskuryakov.

Add support for extracting the OpenSource and Internal revision numbers when Buildbot
returns a dictionary for the value of property got_revision.

* BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotIteration.js:
(parseRevisionProperty): Added; parses Buildbot revision property, which can either be a
string or a dictionary.
(BuildbotIteration.prototype.update.): Modified to call parseRevisionProperty() to determine
the OpenSource and Internal revision numbers.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@162846 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotIteration.js
Tools/ChangeLog

index 2adee8a..cd0c317 100644 (file)
@@ -62,6 +62,17 @@ BuildbotIteration.Event = {
     Updated: "updated"
 };
 
+function parseRevisionProperty(property, key)
+{
+    if (!property)
+        return null;
+
+    // The property "got_revision" is a dictionary for multi-codebase builds
+    // according to <http://docs.buildbot.net/0.8.8/manual/cfg-properties.html>.
+    var value = property[1];
+    return parseInt(typeof value == "string" ? value : value[key], 10);
+}
+
 BuildbotIteration.prototype = {
     constructor: BuildbotIteration,
     __proto__: BaseObject.prototype,
@@ -201,10 +212,10 @@ BuildbotIteration.prototype = {
                 return;
 
             var openSourceRevisionProperty = data.properties.findFirst(function(property) { return property[0] === "got_revision" || property[0] === "revision" || property[0] === "opensource_got_revision"; });
-            this.openSourceRevision = openSourceRevisionProperty ? parseInt(openSourceRevisionProperty[1], 10) : null;
+            this.openSourceRevision = parseRevisionProperty(openSourceRevisionProperty, "WebKitOpenSource");
 
-            var internalRevisionProperty = data.properties.findFirst(function(property) { return property[0] === "internal_got_revision"; });
-            this.internalRevision = internalRevisionProperty ? parseInt(internalRevisionProperty[1], 10) : null;
+            var internalRevisionProperty = data.properties.findFirst(function(property) { return property[0] === "internal_got_revision" || property[0] === "got_revision"; });
+            this.internalRevision = parseRevisionProperty(internalRevisionProperty, "Internal");
 
             var layoutTestResults = collectTestResults.call(this, data, "layout-test");
             this.layoutTestResults = layoutTestResults ? new BuildbotTestResults(this, layoutTestResults) : null;
index a4684ce..b4aa256 100644 (file)
@@ -1,3 +1,19 @@
+2014-01-27  Daniel Bates  <dabates@apple.com>
+
+        Teach Buildbot dashboard to parse alternative revision format
+        https://bugs.webkit.org/show_bug.cgi?id=127561
+
+        Reviewed by Alexey Proskuryakov.
+
+        Add support for extracting the OpenSource and Internal revision numbers when Buildbot
+        returns a dictionary for the value of property got_revision.
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotIteration.js:
+        (parseRevisionProperty): Added; parses Buildbot revision property, which can either be a
+        string or a dictionary.
+        (BuildbotIteration.prototype.update.): Modified to call parseRevisionProperty() to determine
+        the OpenSource and Internal revision numbers.
+
 2014-01-27  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Implement custom URI schemes with CustomProtocols