SoM should not show "Findit is analyzing" alerts on non-chromium trees like ios/etc |
||||
Issue descriptionFindit currently only supports chromium tree. So we'd better not show "Findit is analyzing" for ios/perf/android trees. Otherwise, it would be confusing or misleading sheriffs.
,
Mar 21 2017
,
Mar 29 2017
,
Mar 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/6d804d8c28ed31b6d0fffa7bef97f8cba5c791b8 commit 6d804d8c28ed31b6d0fffa7bef97f8cba5c791b8 Author: Chan <chanli@chromium.org> Date: Thu Mar 30 16:52:38 2017 [SoM-Findit] Only display Findit result for chromium tree and add findit link to running analyses. Bug:702813, 706417 Change-Id: I4df82ce6b306c63672ebc32e7971ac69678afa04 Reviewed-on: https://chromium-review.googlesource.com/462359 Reviewed-by: Sean McCullough <seanmccullough@chromium.org> Commit-Queue: Chan Li <chanli@chromium.org> [modify] https://crrev.com/6d804d8c28ed31b6d0fffa7bef97f8cba5c791b8/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure/som-extension-build-failure.js [modify] https://crrev.com/6d804d8c28ed31b6d0fffa7bef97f8cba5c791b8/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure/som-extension-build-failure.html [modify] https://crrev.com/6d804d8c28ed31b6d0fffa7bef97f8cba5c791b8/go/src/infra/monitoring/analyzer/analyzer_test.go [modify] https://crrev.com/6d804d8c28ed31b6d0fffa7bef97f8cba5c791b8/go/src/infra/monitoring/analyzer/analyzer.go
,
Apr 3 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by chanli@chromium.org
, Mar 17 2017Status: Assigned (was: Available)