| Summary: | [ews-build] Update queues configurations | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | aakash_jain, ap, commit-queue, dewei_zhu, jbedard, lforschler, ryanhaddad, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Other | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Aakash Jain
2019-03-21 06:15:20 PDT
Created attachment 365541 [details]
Patch
Comment on attachment 365541 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=365541&action=review > Tools/BuildSlaveSupport/ews-build/config.json:438 > + "name": "disabled", Assigned few queues which will not be in production initially to another scheduler. This scheduler wouldn't be triggered automatically. Comment on attachment 365541 [details]
Patch
The change looks good to me.
Comment on attachment 365541 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=365541&action=review > Tools/ChangeLog:7 > + I see some platforms moving from iOS 11 to 12. It would be much easier to read this change if you could leave a high level description of this change. > Tools/BuildSlaveSupport/ews-build/config.json:233 > + "name": "ews170", Some of the bots are not used in this patch yet, correct? > Some of the bots are not used in this patch yet, correct?
Yeah, added some bots here which would be configured in future in order to make it easier to configure them.
Committed r243334: <https://trac.webkit.org/changeset/243334> > It would be much easier to read this change if you could leave a high level description of this change.
Updated ChangeLog before landing.
|