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

Issue 655234 link

Starred by 2 users
Status: Verified
Owner:
chanli@chromium.org
Closed: Nov 2016
Cc:
jam@chromium.org
█ robert...@chromium.org
chanli@chromium.org
lijeffrey@chromium.org
seanmccullough@chromium.org
liaoyuke@chromium.org
st...@chromium.org
zhangtiff@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Need to show Findit analysis status on Sheriff-o-Matic UI and give a link back to Findit

Project Member Reported by st...@chromium.org, Oct 12 2016 
On old Sheriff-o-Matic UI, we added a link back to Findit when a culprit is found.
But on the new Sheriff-o-Matic UI, there is no link back to Findit anymore.

We need to add progressive status with links back to Findit.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 28 2016 

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

commit 14ebea953e86d7f76b7c31096e472e151df1d37a
Author: Chan Li <chanli@google.com>
Date: Fri Oct 28 00:26:26 2016

[Som-Findit integration] Add more information from Findit to SoM

1. Added status of Findit analysis
2. For each suspected-cl added confidence score and analysis approaches
3. Made some extra change to surface Findit results
BUG= 655234 

Change-Id: I8d8dfd503ac2b5d474fdfc5c0def0ac6c6a12736
Reviewed-on: https://chromium-review.googlesource.com/402014
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Stephen Martinis <martiniss@chromium.org>

[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/appengine/sheriff-o-matic/elements/pages/som-help.html
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure.html
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/appengine/sheriff-o-matic/elements/som-rev-range.html
[add] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/appengine/sheriff-o-matic/images/README.md
[add] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/appengine/sheriff-o-matic/images/findit.png
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/monitoring/analyzer/analyzer.go
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/monitoring/analyzer/analyzer_test.go
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/monitoring/analyzer/step/test_step.go
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/monitoring/analyzer/step/test_step_test.go
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/monitoring/messages/alerts.go
[modify] https://crrev.com/14ebea953e86d7f76b7c31096e472e151df1d37a/go/src/infra/monitoring/messages/findit.go

Comment 2 by seanmccullough@google.com, Oct 31 2016

Components: Infra>Sheriffing>SheriffOMatic
Labels: Milestone-Workflow
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 1 2016 

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

commit 3e2fb96235a9498bcaef9e287c8d2751c418422b
Author: Stephen Martinis <martiniss@chromium.org>
Date: Mon Oct 31 23:25:00 2016

Update relnotes for SOM

BUG= 655234 , 655286 , 637006 

Change-Id: I914008d6cb9e0c684ceaa53552b57e4700149f38
Reviewed-on: https://chromium-review.googlesource.com/405827
Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org>
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>

[modify] https://crrev.com/3e2fb96235a9498bcaef9e287c8d2751c418422b/go/src/infra/appengine/sheriff-o-matic/RELNOTES.md

Comment 4 by chanli@chromium.org, Nov 14 2016

Status: Verified (was: Assigned)
►

Sign in to add a comment