commit | 4bda7fc4d6183af0855aaf39f1b1a1351f8e0f6a | [log] [tgz] |
---|---|---|
author | Stephen Martinis <martiniss@chromium.org> | Fri May 24 00:51:24 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Fri May 24 00:51:24 2019 |
tree | db848f97c306cc97484faaf55a8f5ea10f6f0da9 | |
parent | e2d9129d0f321097e96d828dae7a979f461c9a34 [diff] |
chromium_swarming: Return has_valid_results Swarming now indicates if a swarming task has valid results. A task can have invalid results if the execution ended improperly. This can be for a few reasons, including the task timing out or swarming having an internal failure. The chromium recipe now uses this information to invalidate test results if swarming says the test has invalid results. This used to be done via throwing exceptions, but this behavior was changed to not do control flow via exceptions, and in the process this behavior appears to have been lost. Recipe-Manual-Change: build_limited_scripts_slave Bug: 965508 Change-Id: Iac2c06dd617345eb22ccdec634276b33f78a5ed7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/1623552 Commit-Queue: Stephen Martinis <martiniss@chromium.org> Reviewed-by: Erik Chen <erikchen@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.