[webkit-reviews] review canceled: [Bug 222540] [Tools] Make run-buildbot-test compatible with buildbot 2.10.5 : [Attachment 425570] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Apr 14 09:08:12 PDT 2021
Carlos Alberto Lopez Perez <clopez at igalia.com> has canceled Carlos Alberto
Lopez Perez <clopez at igalia.com>'s request for review:
Bug 222540: [Tools] Make run-buildbot-test compatible with buildbot 2.10.5
https://bugs.webkit.org/show_bug.cgi?id=222540
Attachment 425570: Patch
https://bugs.webkit.org/attachment.cgi?id=425570&action=review
--- Comment #5 from Carlos Alberto Lopez Perez <clopez at igalia.com> ---
Comment on attachment 425570
--> https://bugs.webkit.org/attachment.cgi?id=425570
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=425570&action=review
>> Tools/CISupport/ews-build/loadConfig.py:103
>> + c['schedulers'].append(forceScheduler)
>
> This shouldn't be enabled in production EWS (atleast for now). Should be
under "if is_test_mode_enabled:"
mmm, right
>> Tools/CISupport/ews-build/steps.py:319
>> + shell.ShellCommand.start(self)
>
> Is this part tested (locally)?
Yes. I assume wget is a command available on any developer machine.
>> Tools/CISupport/start-buildbot-server-virtualenv.py:166
>> + for worker in self._get_list_workers():
>
> Instead of creating ~100 workers on local machine, it's better to use create
just one worker named 'local-worker'. We already have support for
'local-worker' in EWS configuration and I am adding the support in
build.webkit.org in https://bugs.webkit.org/show_bug.cgi?id=224551.
local-worker attaches to all the builders. It is enabled automatically in test
mode (when the BUILDBOT_PRODUCTION env variable is not explicitly set). See
https://github.com/WebKit/WebKit/blob/main/Tools/CISupport/ews-build/loadConfig
.py#L56
Interesting stuff. Will try to use it, thanks.
More information about the webkit-reviews
mailing list