Show reverting CL on SoM |
||
Issue descriptionWe need to figure out what info and how to show on SoM.
,
Mar 13 2017
Possible way to display the reverting CL: 1. Add a sentence after Findit result: findit Findit (?) found culprit 0510799 with 100% confidence. More details... Reverting CL (url) has been created, PLEASE CHECK. 2. Add a header under tree close: Tree is closed ... Findit created a reverting CL(url) for the culprit (rev) of this tree close. PLEASE CHECK(or CONFIRM).
,
Mar 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/8de90094826b34437113a6356efce9273432e691 commit 8de90094826b34437113a6356efce9273432e691 Author: Chan <chanli@chromium.org> Date: Wed Mar 15 18:16:45 2017 [Findit] Add reverting CL url to the response of Findit API. BUG= 700636 Change-Id: Ieeeee7f73d398a0111c312a1f7110e552ad62f50 Reviewed-on: https://chromium-review.googlesource.com/453885 Reviewed-by: Shuotao Gao <stgao@chromium.org> Reviewed-by: Jeffrey Li <lijeffrey@chromium.org> Commit-Queue: Chan Li <chanli@chromium.org> [modify] https://crrev.com/8de90094826b34437113a6356efce9273432e691/appengine/findit/test/findit_api_test.py [modify] https://crrev.com/8de90094826b34437113a6356efce9273432e691/appengine/findit/findit_api.py
,
Mar 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/04db64bcabbe707ae5b26586955de440db41899b commit 04db64bcabbe707ae5b26586955de440db41899b Author: Chan <chanli@chromium.org> Date: Wed Mar 15 23:44:38 2017 [SoM-Findit] Add reverting CL to Findit results on Sheriff-o-Matic. After Findit gets the url to the reverting CL of a culprit, it will send this info to Sheriff-o-Matic. Currently only add a sentence after Findit result as "Reverting CL (url) has been created. PLEASE CHECK." Design doc: https://docs.google.com/a/google.com/document/d/1_nrlNP8kIKvl5XgoBX0k8TOwNypkK-xmZ89C0UdqEv0/edit?usp=sharing BUG= 700636 Change-Id: I554491e7bc27e499ca09bdf1f2075c4cf4a401d2 Reviewed-on: https://chromium-review.googlesource.com/453985 Reviewed-by: Sean McCullough <seanmccullough@chromium.org> Commit-Queue: Chan Li <chanli@chromium.org> [modify] https://crrev.com/04db64bcabbe707ae5b26586955de440db41899b/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure/som-extension-build-failure.js [modify] https://crrev.com/04db64bcabbe707ae5b26586955de440db41899b/go/src/infra/monitoring/messages/findit.go [modify] https://crrev.com/04db64bcabbe707ae5b26586955de440db41899b/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure/som-extension-build-failure.html [modify] https://crrev.com/04db64bcabbe707ae5b26586955de440db41899b/go/src/infra/monitoring/analyzer/analyzer_test.go
,
Mar 16 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by chanli@chromium.org
, Mar 13 2017Status: Started (was: Assigned)