| Index: scripts/slave/recipe_modules/auto_bisect/revision_state.py
|
| diff --git a/scripts/slave/recipe_modules/auto_bisect/revision_state.py b/scripts/slave/recipe_modules/auto_bisect/revision_state.py
|
| index 3b85b48d8eb97a3d91084541cb3e4c735a6a5d52..24cb7f5fe404845449c843451ea68fc26414cca6 100644
|
| --- a/scripts/slave/recipe_modules/auto_bisect/revision_state.py
|
| +++ b/scripts/slave/recipe_modules/auto_bisect/revision_state.py
|
| @@ -31,10 +31,7 @@
|
| SKIPPED, # A revision that was not built or tested for a special reason,
|
| # such as those ranges that we know are broken, or when nudging
|
| # revisions.
|
| - NEED_MORE_DATA, # Current number of test values is too small to establish
|
| - # a statistically significant difference between this
|
| - # revision and the revisions known to be good and bad.
|
| - ) = xrange(8)
|
| + ) = xrange(7)
|
|
|
| def __init__(self, revision_string, bisector,
|
| dependency_depot_name=None, base_revision=None,
|
| @@ -114,12 +111,11 @@
|
|
|
| @property
|
| def tested(self):
|
| - return self.status in (RevisionState.TESTED,)
|
| + return self.status in [RevisionState.TESTED]
|
|
|
| @property
|
| def in_progress(self):
|
| - return self.status in (RevisionState.BUILDING, RevisionState.TESTING,
|
| - RevisionState.NEED_MORE_DATA)
|
| + return self.status in (RevisionState.BUILDING, RevisionState.TESTING)
|
|
|
| @property
|
| def failed(self):
|
| @@ -152,9 +148,8 @@
|
| self.status = RevisionState.BUILDING
|
| return
|
|
|
| - if self._is_build_archived() and self.status in (
|
| - RevisionState.NEW, RevisionState.BUILDING,
|
| - RevisionState.NEED_MORE_DATA):
|
| + if self._is_build_archived() and self.status in [RevisionState.NEW,
|
| + RevisionState.BUILDING]:
|
| self._do_test()
|
| self.status = RevisionState.TESTING
|
|
|
| @@ -252,16 +247,11 @@
|
| """
|
| if self.status == RevisionState.BUILDING and self._is_build_archived():
|
| self.start_job()
|
| - elif (self.status in (RevisionState.TESTING, RevisionState.NEED_MORE_DATA)
|
| - and self._results_available()):
|
| - # If we have already decided whether the revision is good or bad we
|
| - # shouldn't check again
|
| - check_revision_goodness = not(self.good or self.bad)
|
| - self._read_test_results(
|
| - check_revision_goodness=check_revision_goodness)
|
| + elif self.status == RevisionState.TESTING and self._results_available():
|
| + self._read_test_results()
|
| # We assume _read_test_results may have changed the status to a broken
|
| # state such as FAILED or ABORTED.
|
| - if self.status in (RevisionState.TESTING, RevisionState.NEED_MORE_DATA):
|
| + if self.status == RevisionState.TESTING:
|
| self.status = RevisionState.TESTED
|
|
|
| def _is_build_archived(self):
|
|
|