New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 703368 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 700627



Sign in to add a comment

[Findit-SoM] Add one more criteria (Findit results) to SoM when sorting alerts in each section

Project Member Reported by chanli@chromium.org, Mar 20 2017

Issue description

Add a check of whether the alert has Findit result or not to the sorting criteria to SoM alerts.
 

Comment 1 by chanli@chromium.org, Mar 20 2017

Blocking: 700627
Labels: Milestone-Workflow
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 23 2017

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

commit e0a819ec0b72edd4845cc0f4b25fa131bc11ae28
Author: Chan <chanli@chromium.org>
Date: Thu Mar 23 18:48:54 2017

[SoM-Findit] Sort alerts to display the ones with Findit results on top.

At each section:
1. Display alerts with revert CL from Findit first;
2. then alerts with suspected cls;
3. then alerts with confirmed flaky tests;
4. then alerts with no Findit findings.

BUG= 703368 

Change-Id: Icae93d03827100f4bab5c32db87aa3e7b24e0c4b
Reviewed-on: https://chromium-review.googlesource.com/457440
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Chan Li <chanli@chromium.org>

[modify] https://crrev.com/e0a819ec0b72edd4845cc0f4b25fa131bc11ae28/go/src/infra/appengine/sheriff-o-matic/elements/som-app/som-app.js
[modify] https://crrev.com/e0a819ec0b72edd4845cc0f4b25fa131bc11ae28/go/src/infra/appengine/sheriff-o-matic/test/som-app-test.html

Comment 4 by chanli@chromium.org, Mar 23 2017

Status: Fixed (was: Assigned)

Sign in to add a comment