Audit alert has multiple entries for the same rule. |
||
Issue descriptionLook at bug 767325 for example, what happened is that the first time the audit ran, it failed, but it saved a partial result that the second run only appended to.
,
Sep 21 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/62a1cd3454dae85cb3b7bd1c4c0a5a9491202c3d commit 62a1cd3454dae85cb3b7bd1c4c0a5a9491202c3d Author: Roberto Carrillo <robertocn@google.com> Date: Thu Sep 21 20:08:43 2017 [cr-audit-commits] Discard partial results on panic. If a rule panics while auditing a commit, do not save the results of the previous rules so that a rerun doesn't append repeated entries to this incomplete list. R=stgao,chanli BUG= 767535 Change-Id: Ic93117e622c9b48b96ad0aff32c3ec4860c92557 Reviewed-on: https://chromium-review.googlesource.com/677572 Reviewed-by: Shuotao Gao <stgao@chromium.org> Commit-Queue: Roberto Carrillo <robertocn@chromium.org> [modify] https://crrev.com/62a1cd3454dae85cb3b7bd1c4c0a5a9491202c3d/go/src/infra/appengine/cr-audit-commits/app/commit_auditor.go
,
Sep 21 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by robert...@chromium.org
, Sep 21 2017