Handle regression ranges, test results, and findit results on grouped alerts |
|||
Issue descriptionIt looks like these particular parts of Sheriff-o-Matic aren't really things that Chrome OS uses, so we'll have to get these to work with alert grouping. Simplest would be to just make these things show up on the individual subalerts in a group. It gets a little trickier if we want to display these things in the merged view of all the alerts in a group.
,
Aug 25 2017
,
Sep 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/67f4ad368c01f00fd326492e6d0da22a71bef7ce commit 67f4ad368c01f00fd326492e6d0da22a71bef7ce Author: Stephen Martinis <martiniss@chromium.org> Date: Thu Sep 07 22:02:12 2017 SOM: Group regression ranges correctly Bug:723875 Change-Id: I31b055e6196a76cd10b4c4a9f33862573ea60baf Reviewed-on: https://chromium-review.googlesource.com/636176 Reviewed-by: Sean McCullough <seanmccullough@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> [modify] https://crrev.com/67f4ad368c01f00fd326492e6d0da22a71bef7ce/go/src/infra/appengine/sheriff-o-matic/frontend/elements/som-alert-view/som-alert-view.js [modify] https://crrev.com/67f4ad368c01f00fd326492e6d0da22a71bef7ce/go/src/infra/appengine/sheriff-o-matic/frontend/test/som-alert-view-test.html
,
Oct 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/ef0b4313e7518f16c109b08b2a674fa81669974f commit ef0b4313e7518f16c109b08b2a674fa81669974f Author: Stephen Martinis <martiniss@chromium.org> Date: Mon Oct 30 20:28:41 2017 SOM: Fix reason merging There's a small logic bug in the code, where if a group has a reason, and there's nothing wrong with merging in the new alert's reason, it won't actually merge the reason. This CL fixes this, and adds a test. Bug: 723875 Change-Id: Iff771b5fe6800f3aca1b7fcfbbe9926d89d96f5b Reviewed-on: https://chromium-review.googlesource.com/730882 Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> [modify] https://crrev.com/ef0b4313e7518f16c109b08b2a674fa81669974f/go/src/infra/appengine/sheriff-o-matic/frontend/elements/som-alert-view/som-alert-view.js [modify] https://crrev.com/ef0b4313e7518f16c109b08b2a674fa81669974f/go/src/infra/appengine/sheriff-o-matic/frontend/test/som-alert-view-test.html
,
Dec 14 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by davidri...@chromium.org
, Jun 5 2017