[webkit-reviews] review granted: [Bug 202361] [GTK] Add EWS testers to run GTK unit tests : [Attachment 388638] Patch

[bugzilla-daemon at webkit.org]

Aakash Jain <aakash_jain at apple.com> has granted Carlos Alberto Lopez Perez
<clopez at igalia.com>'s request for review:
Bug 202361: [GTK] Add EWS testers to run GTK unit tests
https://bugs.webkit.org/show_bug.cgi?id=202361

Attachment 388638: Patch

https://bugs.webkit.org/attachment.cgi?id=388638&action=review




--- Comment #28 from Aakash Jain <aakash_jain at apple.com> ---
Comment on attachment 388638
  --> https://bugs.webkit.org/attachment.cgi?id=388638
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=388638&action=review

> Tools/BuildSlaveSupport/ews-build/config.json:-318
> -	 "workernames": ["igalia1-gtk-wk2-ews", "igalia2-gtk-wk2-ews",
"igalia3-gtk-wk2-ews", "igalia4-gtk-wk2-ews", "aperez-gtk-ews"]

Do you want to remove "igalia3-gtk-wk2-ews" from GTK-Webkit2-EWS queue. Would 2
bots be sufficient for this queue? You have the option of keeping a bot in both
the queues, and share the bots on both queues (e.g.: ews151, ews119 are shared
on multiple queues).