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>
6 files changed
tree: db848f97c306cc97484faaf55a8f5ea10f6f0da9
  1. .gitattributes
  2. .gitignore
  3. .vpython
  4. DEPS
  5. LICENSE
  6. OWNERS
  7. PRESUBMIT.py
  8. README.md
  9. WATCHLISTS
  10. codereview.settings
  11. environment.cfg.py
  12. infra/
  13. masters/
  14. scripts/
  15. site_config/
  16. slave/
  17. tests/
  18. third_party/
README.md

build

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.

Recipes

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.