Attachment 347038 Details for Bug 188531 – Proposed patch

[patch] Proposed patch

ews_max_builds.patch (text/plain), 1.46 KB, created by Aakash Jain on 2018-08-13 14:31:58 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Aakash Jain

Created: 2018-08-13 14:31:58 PDT

Size: 1.46 KB

patch

obsolete

>Index: Tools/ChangeLog
>===================================================================
>--- Tools/ChangeLog	(revision 234817)
>+++ Tools/ChangeLog	(working copy)
>@@ -1,3 +1,13 @@
>+2018-08-13  Aakash Jain  <aakash_jain@apple.com>
>+
>+        [ews-build] Add support for max_builds parameter for workers
>+        https://bugs.webkit.org/show_bug.cgi?id=188531
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * BuildSlaveSupport/ews-build/loadConfig.py:
>+        (loadBuilderConfig): Added support for max_builds.
>+
> 2018-08-13  Wenson Hsieh  <wenson_hsieh@apple.com>
> 
>         [WK2] [macOS] Implement a mechanism to test drag and drop
>Index: Tools/BuildSlaveSupport/ews-build/loadConfig.py
>===================================================================
>--- Tools/BuildSlaveSupport/ews-build/loadConfig.py	(revision 234790)
>+++ Tools/BuildSlaveSupport/ews-build/loadConfig.py	(working copy)
>@@ -43,7 +43,7 @@ def loadBuilderConfig(c, use_localhost_w
>     checkWorkersAndBuildersForConsistency(config, config['workers'], config['builders'])
>     checkValidSchedulers(config, config['schedulers'])
> 
>-    c['workers'] = [Worker(worker['name'], passwords.get(worker['name'], 'password')) for worker in config['workers']]
>+    c['workers'] = [Worker(worker['name'], passwords.get(worker['name'], 'password'), max_builds=worker.get('max_builds', 1)) for worker in config['workers']]
>     if use_localhost_worker:
>         c['workers'].append(Worker('local-worker', 'password', max_builds=2))
>

Actions: View | Formatted Diff | Diff

Attachments on bug 188531: 347038