Have parseRevisionProperty use default repo name as the Buildbot key.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Apr 2017 03:33:56 +0000 (03:33 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Apr 2017 03:33:56 +0000 (03:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=170696

Patch by Kocsen Chung <kocsen_chung@apple.com> on 2017-04-10
Reviewed by Daniel Bates.

Previously, we needed to set a key and a fallbackKey when addressing
Buildbot data for Internal. That is no longer necessary as we can make a safe assumption
that the repo name maps exactly to that key for every repository other
than WebKit.

* BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotIteration.js:
Set `key = repositoryName` and only use fallback keys when dealing with WebKit.

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

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

index 63e4935..83ac211 100644 (file)
@@ -201,21 +201,21 @@ BuildbotIteration.prototype = {
             return property[0] === "got_revision";
         });
         var branches = this.queue.branches;
+
         for (var i = 0; i < branches.length; ++i) {
             var repository = branches[i].repository;
             var repositoryName = repository.name;
             var key;
             var fallbackKey;
+
             if (repository === Dashboard.Repository.OpenSource) {
                 key = "WebKit";
                 fallbackKey = "opensource";
-            } else if (repository === Dashboard.Repository.Internal) {
-                key = "Internal";
-                fallbackKey = "internal";
             } else {
                 key = repositoryName;
                 fallbackKey = null;
             }
+
             var revision = parseRevisionProperty(revisionProperty, key, fallbackKey);
             this.revision[repositoryName] = revision;
         }
index 7becd66..224c05c 100644 (file)
@@ -1,3 +1,18 @@
+2017-04-10  Kocsen Chung  <kocsen_chung@apple.com>
+
+        Have parseRevisionProperty use default repo name as the Buildbot key.
+        https://bugs.webkit.org/show_bug.cgi?id=170696
+
+        Reviewed by Daniel Bates.
+
+        Previously, we needed to set a key and a fallbackKey when addressing
+        Buildbot data for Internal. That is no longer necessary as we can make a safe assumption
+        that the repo name maps exactly to that key for every repository other
+        than WebKit.
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotIteration.js:
+        Set `key = repositoryName` and only use fallback keys when dealing with WebKit.
+
 2017-04-10  Aakash Jain  <aakash_jain@apple.com>
 
         Add test262 JavaScriptCore tests to dashboard