New issue
Advanced search Search tips

Issue 780297 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Split up alerts per test in sheriff-o-matic

Project Member Reported by martiniss@chromium.org, Oct 31 2017

Issue description

Alerts which have test results should be split up per test, rather than having one alert per test case.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/f47e08fb20a5f6e418a9e7be899ab25fc0a4762e

commit f47e08fb20a5f6e418a9e7be899ab25fc0a4762e
Author: Stephen Martinis <martiniss@chromium.org>
Date: Wed Nov 01 01:07:40 2017

SOM: Split alerts by test

This CL splits up alerts per test case in a test suite. Previous CLs
created the ability to have more flexible groups in the frontend. This CL
splits up alerts with test results data into individual alerts per test
failure. These alerts will then be grouped per test case, rather than
test suite.

Bug:780297
Change-Id: Iaf94b1ae8740cc5826a84c991ed7d3b44d9cc397
Reviewed-on: https://chromium-review.googlesource.com/671157
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>

[modify] https://crrev.com/f47e08fb20a5f6e418a9e7be899ab25fc0a4762e/go/src/infra/appengine/sheriff-o-matic/som/analyzer/analyzer.go
[modify] https://crrev.com/f47e08fb20a5f6e418a9e7be899ab25fc0a4762e/go/src/infra/appengine/sheriff-o-matic/som/analyzer/step/test_step_test.go
[modify] https://crrev.com/f47e08fb20a5f6e418a9e7be899ab25fc0a4762e/go/src/infra/appengine/sheriff-o-matic/som/analyzer/step/test_step.go

Labels: Milestone-Workflow
Status: Fixed (was: Assigned)

Sign in to add a comment