diff options
author | Caroline Tice <cmtice@google.com> | 2016-11-30 09:41:28 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2016-12-01 17:06:08 -0800 |
commit | c7ee80e420941341fedaf8e620776f5b4a9df9df (patch) | |
tree | 9ae5d46c19ef807db839a13a0a22757062c117d9 /binary_search_tool/binary_search_state.py | |
parent | e02e9f819648419cc1d6e59d02ddf50d16f0ca22 (diff) | |
download | toolchain-utils-c7ee80e420941341fedaf8e620776f5b4a9df9df.tar.gz |
[binary search tool] Allow non-flashing images to be fails.
Currently if an image is so bad that it can't be flashed, it gets
marked as 'skipped', which is different from bad/failed. This can
cause the binary search to never find the bad image. We need an
option to treat a non-flashing image as a failure. That's what
this CL does.
BUG=None
TEST=Tested by Manoj: without this change, binary search failed to
find any bad packages, but with this change it worked.
Change-Id: Ib223338a1820314e85352827857fb8f3ba1ed73a
Reviewed-on: https://chrome-internal-review.googlesource.com/307824
Commit-Ready: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Reviewed-by: Manoj Gupta <manojgupta@google.com>
Diffstat (limited to 'binary_search_tool/binary_search_state.py')
-rwxr-xr-x | binary_search_tool/binary_search_state.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/binary_search_tool/binary_search_state.py b/binary_search_tool/binary_search_state.py index 8356d93e..a10e90b9 100755 --- a/binary_search_tool/binary_search_state.py +++ b/binary_search_tool/binary_search_state.py @@ -293,9 +293,6 @@ class BinarySearchState(object): status = self.TestSetupScript() if status == 0: status = self.TestScript() - else: - # Test setup script failed, treat as skipped item - status = 125 terminated = self.binary_search.SetStatus(status) if terminated: |