[webkit-changes] [221717] trunk/Websites/perf.webkit.org

Wed, 06 Sep 2017 21:24:08 -0700

Title: [221717] trunk/Websites/perf.webkit.org
Revision
221717
Author
aakash_j...@apple.com
Date
2017-09-06 21:23:35 -0700 (Wed, 06 Sep 2017)

Log Message

Add initSyncers method in BuildbotTriggerable
https://bugs.webkit.org/show_bug.cgi?id=176125

Reviewed by Ryosuke Niwa.

* tools/sync-buildbot.js:
(syncLoop): Use initSyncers() which returns a promise. Modified to handle the promise.
* tools/js/buildbot-triggerable.js:
(BuildbotTriggerable): Invokes initSyncers() appropriately.
(BuildbotTriggerable.prototype.initSyncers): Returns a promise which initialize all the syncers.
* server-tests/tools-buildbot-triggerable-tests.js: Updated tests to handle initSyncers().
* server-tests/tools-sync-buildbot-integration-tests.js: Ditto.

Modified Paths

Diff

Modified: trunk/Websites/perf.webkit.org/ChangeLog (221716 => 221717)


--- trunk/Websites/perf.webkit.org/ChangeLog	2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/ChangeLog	2017-09-07 04:23:35 UTC (rev 221717)
@@ -1,3 +1,18 @@
+2017-09-06  Aakash Jain  <aakash_j...@apple.com>
+
+        Add initSyncers method in BuildbotTriggerable
+        https://bugs.webkit.org/show_bug.cgi?id=176125
+
+        Reviewed by Ryosuke Niwa.
+
+        * tools/sync-buildbot.js:
+        (syncLoop): Use initSyncers() which returns a promise. Modified to handle the promise.
+        * tools/js/buildbot-triggerable.js:
+        (BuildbotTriggerable): Invokes initSyncers() appropriately.
+        (BuildbotTriggerable.prototype.initSyncers): Returns a promise which initialize all the syncers.
+        * server-tests/tools-buildbot-triggerable-tests.js: Updated tests to handle initSyncers().
+        * server-tests/tools-sync-buildbot-integration-tests.js: Ditto.
+
 2017-09-05  Ryosuke Niwa  <rn...@webkit.org>
 
         Add a button to show two weeks of data to perf dashboard

Modified: trunk/Websites/perf.webkit.org/server-tests/tools-buildbot-triggerable-tests.js (221716 => 221717)


--- trunk/Websites/perf.webkit.org/server-tests/tools-buildbot-triggerable-tests.js	2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/server-tests/tools-buildbot-triggerable-tests.js	2017-09-07 04:23:35 UTC (rev 221717)
@@ -35,7 +35,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(BuildRequest.all().length, 4);
@@ -89,7 +89,7 @@
                 let logger = new MockLogger;
                 let slaveInfo = {name: 'sync-slave', password: 'password'};
                 let triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests[0].method, 'GET');
@@ -137,7 +137,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 2);
@@ -216,7 +216,7 @@
                 let logger = new MockLogger;
                 let slaveInfo = {name: 'sync-slave', password: 'password'};
                 let triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 2);
@@ -288,7 +288,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 2);
@@ -367,7 +367,7 @@
                 let logger = new MockLogger;
                 let slaveInfo = {name: 'sync-slave', password: 'password'};
                 let triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 1);
@@ -430,7 +430,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 2);
@@ -528,7 +528,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 2);
@@ -629,7 +629,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(requests.length, 1);
@@ -704,7 +704,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(requests.length, 1);
@@ -731,7 +731,7 @@
                     {[-1]: MockData.runningBuild({buildRequestId: 700}), [-2]: MockData.finishedBuild({buildRequestId: 710})});
                 return syncPromise;
             }).then(() => {
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(requests.length, 6);
@@ -764,7 +764,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 1);
@@ -831,7 +831,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 1);
@@ -886,7 +886,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                syncPromise = triggerable.syncOnce();
+                syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
                 return MockRemoteAPI.waitForRequest();
             }).then(() => {
                 assert.equal(MockRemoteAPI.requests.length, 1);
@@ -972,7 +972,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const buildbotTriggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                return buildbotTriggerable.updateTriggerable();
+                return buildbotTriggerable.initSyncers().then(() => buildbotTriggerable.updateTriggerable());
             }).then(() => refetchManifest()).then(() => {
                 assert.equal(Triggerable.all().length, 1);
 
@@ -997,7 +997,7 @@
                 const logger = new MockLogger;
                 const slaveInfo = {name: 'sync-slave', password: 'password'};
                 const buildbotTriggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
-                return buildbotTriggerable.updateTriggerable();
+                return buildbotTriggerable.initSyncers().then(() => buildbotTriggerable.updateTriggerable());
             }).then(() => refetchManifest()).then(() => {
                 assert.equal(Triggerable.all().length, 1);
                 const groups = TriggerableRepositoryGroup.sortByName(Triggerable.all()[0].repositoryGroups());

Modified: trunk/Websites/perf.webkit.org/server-tests/tools-sync-buildbot-integration-tests.js (221716 => 221717)


--- trunk/Websites/perf.webkit.org/server-tests/tools-sync-buildbot-integration-tests.js	2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/server-tests/tools-sync-buildbot-integration-tests.js	2017-09-07 04:23:35 UTC (rev 221717)
@@ -61,7 +61,7 @@
         return Manifest.fetch();
     }).then(() => {
         triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, {name: 'sync-slave', password: 'password'}, new MockLogger);
-        return triggerable.updateTriggerable();
+        return triggerable.initSyncers().then(() => triggerable.updateTriggerable());
     }).then(() => Manifest.fetch()).then(() => {
         return new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, {name: 'sync-slave', password: 'password'}, new MockLogger);
     });
@@ -156,7 +156,7 @@
             assert.equal(otherCommitSet.rootForRepository(webkit), null);
             assert.deepEqual(otherCommitSet.allRootFiles(), []);
 
-            syncPromise = triggerable.syncOnce();
+            syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
             return MockRemoteAPI.waitForRequest();
         }).then(() => {
             assert.equal(requests.length, 3);
@@ -267,7 +267,7 @@
             assert.deepEqual(otherCommitSet.allRootFiles(), []);
 
             MockRemoteAPI.reset();
-            syncPromise = triggerable.syncOnce();
+            syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
             return MockRemoteAPI.waitForRequest();
         }).then(() => {
             assert.equal(requests.length, 3);
@@ -441,7 +441,7 @@
             const otherRoots = otherBuildRequest.commitSet().allRootFiles();
             assert.equal(otherRoots.length, 1);
             assert.deepEqual(otherRoots[0].filename(), 'root46.dat');
-            syncPromise = triggerable.syncOnce();
+            syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
             return MockRemoteAPI.waitForRequest();
         }).then(() => {
             assert.equal(requests.length, 3);
@@ -512,7 +512,7 @@
             assert.equal(otherBuildRequest.statusUrl(), null);
             assert.equal(otherBuildRequest.buildId(), null);
 
-            syncPromise = triggerable.syncOnce();
+            syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
             return Promise.all([MockRemoteAPI.waitForRequest(), uploadRoot(1, 123)]);
         }).then(() => {
             assert.equal(requests.length, 3);
@@ -596,7 +596,7 @@
             assert.equal(otherBuildRequest.statusUrl(), null);
             assert.equal(otherBuildRequest.buildId(), null);
 
-            syncPromise = triggerable.syncOnce();
+            syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
             return MockRemoteAPI.waitForRequest();
         }).then(() => {
             assert.equal(requests.length, 3);

Modified: trunk/Websites/perf.webkit.org/tools/js/buildbot-triggerable.js (221716 => 221717)


--- trunk/Websites/perf.webkit.org/tools/js/buildbot-triggerable.js	2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/tools/js/buildbot-triggerable.js	2017-09-07 04:23:35 UTC (rev 221717)
@@ -16,15 +16,25 @@
         assert(typeof(this._lookbackCount) == 'number' && this._lookbackCount > 0, 'lookbackCount must be a number greater than 0');
 
         this._remote = remote;
+        this._config = config;
+        this._buildbotRemote = buildbotRemote;
 
         this._slaveInfo = slaveInfo;
         assert(typeof(slaveInfo.name) == 'string', 'slave name must be specified');
         assert(typeof(slaveInfo.password) == 'string', 'slave password must be specified');
 
-        this._syncers = BuildbotSyncer._loadConfig(buildbotRemote, config);
+        this._syncers = null;
         this._logger = logger || {log: () => { }, error: () => { }};
     }
 
+    initSyncers()
+    {
+        return new Promise((resolve, reject) => {
+            this._syncers = BuildbotSyncer._loadConfig(this._buildbotRemote, this._config);
+            setTimeout(resolve, 0);
+        });
+    }
+
     name() { return this._name; }
 
     updateTriggerable()

Modified: trunk/Websites/perf.webkit.org/tools/sync-buildbot.js (221716 => 221717)


--- trunk/Websites/perf.webkit.org/tools/sync-buildbot.js	2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/tools/sync-buildbot.js	2017-09-07 04:23:35 UTC (rev 221717)
@@ -35,11 +35,13 @@
     };
 
     Manifest.fetch().then(() => {
-        return makeTriggerable().updateTriggerable();
+        const triggerable = makeTriggerable();
+        return triggerable.initSyncers().then(() => triggerable.updateTriggerable());
     }).then(() => {
         return Manifest.fetch();
     }).then(() => {
-        return makeTriggerable().syncOnce();
+        const triggerable = makeTriggerable();
+        return triggerable.initSyncers().then(() => triggerable.syncOnce());
     }).catch((error) => {
         console.error(error);
         if (typeof(error.stack) == 'string') {

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to