commit | ab4a6ed2d7d2210e9c45a111e0af8e53f1c1c266 | [log] [tgz] |
---|---|---|
author | Patrik Höglund <phoglund@chromium.org> | Fri Mar 09 14:27:35 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Mar 09 14:44:24 2018 |
tree | 6b17687574f17d1c38af1dcbdab7febe142a6ea8 | |
parent | 11e4ec1e00bec047aa33682e5cc8e7a5bd380a36 [diff] |
Don't retry perf test failures for WebRTC; re-enable isac_fix. This sends an arg to src/build/android/test_runner.py so it doesn't attempt to retry failed tests. The default is 2 retries. The problem is that webrtc_perf_tests writes a .json, but the android test runner retry mechanism invokes the binary again, with the test filter set to the remaining failed tests. Therefore, the .json is overwritten and the results from the first run are lost. We may want to allow retries later, but then we must slightly re-design perf result handling in the WebRTC binaries. I also re-enable isac_fix_test here since I forgot that earlier. Bug: chromium:755660 Change-Id: Ia6be0bc4138268c33c063173229724ec69f16ee7 Reviewed-on: https://chromium-review.googlesource.com/956046 Commit-Queue: Patrik Höglund <phoglund@chromium.org> Reviewed-by: Edward Lesmes <ehmaldonado@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.