aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/isolate_win_toolchain.isolate
Commit message (Collapse)AuthorAge
* Revert "[infra] Move commands from isolates to gen_tasks.go"Gravatar Eric Boren2017-12-07
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 66db75da67fb4c75207c2e495765e1e74584e3b1. Reason for revert: Breaking PDFium/Flutter Original change's description: > [infra] Move commands from isolates to gen_tasks.go > > This has the side effect of bundling recipes for all tasks, > which will be required anyway to run recipes in the future. > > Bug: skia:7050 > Change-Id: Ia30a95c750f2a237a8bf60263b2981682673c043 > Reviewed-on: https://skia-review.googlesource.com/81300 > Commit-Queue: Eric Boren <borenet@google.com> > Reviewed-by: Kevin Lubick <kjlubick@google.com> TBR=borenet@google.com,bungeman@google.com,rmistry@google.com,kjlubick@google.com Change-Id: I38e00294f098ac91285a5cc464414654fc79a9e9 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia:7050 Reviewed-on: https://skia-review.googlesource.com/81741 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
* [infra] Move commands from isolates to gen_tasks.goGravatar Eric Boren2017-12-07
| | | | | | | | | | | This has the side effect of bundling recipes for all tasks, which will be required anyway to run recipes in the future. Bug: skia:7050 Change-Id: Ia30a95c750f2a237a8bf60263b2981682673c043 Reviewed-on: https://skia-review.googlesource.com/81300 Commit-Queue: Eric Boren <borenet@google.com> Reviewed-by: Kevin Lubick <kjlubick@google.com>
* Isolate the biggest CIPD assets to trim down on overheadGravatar Kevin Lubick2017-11-29
This change, when combined with https://skia-review.googlesource.com/c/buildbot/+/77701 should cut down on the duplicated work of extracting the large toolchains from CIPD. Since the Isolate* steps can be cached (i.e. are idempotent) they will only run about 1/week (unless updated) and all subsequent tasks (primarily Builds) will go much faster. We estimate the overhead on Build-Debian-Android to go from about 90s (which was more time than the actual build) to about 10s. Build-Win-Vulkan's overhead will improve from about 180s to about 35s (1/3 of which is uploading to isolate). Other CIPD assets could be handled in a similar fashion; the ones here are the biggest offenders and the lowest hanging fruit. Doing this to other assets (e.g. clang_win) would have minimal improvements (<10s). There are other tasks with large amounts of overhead (e.g. Build-Mac-Android, Build-Win-Android, Build-WASM) but none of those are depended on by any Tests, so any speed-ups would have less wide-reaching-impact, at the cost of using more Isolate cache/diskspace. See https://docs.google.com/document/d/1DFlcpqg7XqEPE5oYT1V3so2ih2285heS5w3mPT-GMBA/edit# for more information. Bug: skia: Change-Id: I40dd87fe72c3d49292762a09dad6df0dfbe78f61 Reviewed-on: https://skia-review.googlesource.com/77560 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Kevin Lubick <kjlubick@google.com>