Audit auto-reverts for test failures. |
||
Issue descriptionThe audit app should correctly audit auto-reverts created by Findit for test failures. The only change in behavior is to validate that the sample step of the sample failed build is indeed failed (instead of always checking the 'compile' step.)
,
Feb 21 2018
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/ea60b1b1a1ebebe1f6804b0209555efc272ec891 commit ea60b1b1a1ebebe1f6804b0209555efc272ec891 Author: Chan <chanli@chromium.org> Date: Wed Feb 21 01:07:41 2018 [Findit] Add sample failed step in revert reason. Since now we begin to auto-revert culprits of consistent test failures, add a sample failed step name to the revert reason. This can help audit app to better audit Findit's auto-reverted commits. Bug: 813941 Change-Id: Ib5e3f5087bf77edf7174d29b2f5b6e979f6de8e8 Reviewed-on: https://chromium-review.googlesource.com/927783 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Roberto Carrillo <robertocn@chromium.org> [modify] https://crrev.com/ea60b1b1a1ebebe1f6804b0209555efc272ec891/appengine/findit/services/test/culprit_action_test.py [modify] https://crrev.com/ea60b1b1a1ebebe1f6804b0209555efc272ec891/appengine/findit/services/gerrit.py [modify] https://crrev.com/ea60b1b1a1ebebe1f6804b0209555efc272ec891/appengine/findit/services/culprit_action.py [modify] https://crrev.com/ea60b1b1a1ebebe1f6804b0209555efc272ec891/appengine/findit/services/test/gerrit_test.py [modify] https://crrev.com/ea60b1b1a1ebebe1f6804b0209555efc272ec891/appengine/findit/pipelines/test/create_revert_cl_pipeline_test.py
,
Apr 9 2018
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Feb 21 2018