commit | c52a9790db08f46051e02a8da12b77caf3f04a5b | [log] [tgz] |
---|---|---|
author | Stephen Martinis <martiniss@chromium.org> | Fri Jan 11 18:24:16 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Jan 11 18:24:16 2019 |
tree | 066bfc5d9a52003a82f61e8c3f1aa22164cec017 | |
parent | b9dd95df658d21f6b2e3a13f1911e9cafa86cfcf [diff] |
Batch tasks to wait for wait_for_finished_task_set.py currently requests task status from swarming for every task it's supposed to wait for. This is problematic if the script is supposed to wait for hundreds of tasks, as there are limits on URL length in app engine. This CL changes the script to batch the requests into sets of 300 tasks, which is small enough to work. Bug: 920724 Change-Id: Icb3d0169eb3c95c7ac05c2508a1e28b579f4f426 Reviewed-on: https://chromium-review.googlesource.com/c/1405618 Commit-Queue: Stephen Martinis <martiniss@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Auto-Submit: Stephen Martinis <martiniss@chromium.org> Reviewed-by: Garrett Beaty <gbeaty@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Hi build contributor! If you do any change in scripts/master/
or touching any master's html/
directories, you must restart master.chromium.fyi first and ensure that it still works before restarting other masters.
If you're here to make a change to ‘recipes’ (the code located in scripts/slave/recipes*
), please take a look at the README for more information pertaining to recipes.