The "master <x> scheduled for a restart at <time>" doesn't show internal masters |
||||
Issue descriptionI handled three tickets to restart masters, one for chromium.android, one for client.boringssl, and one for chromeos.branch. After filing the three CLs, a nice purple box showed up on the top of /trooper saying that the masters would be restarted. Awesome! Except it didn't have a line saying that chromeos.branch would be restarted. It only showed the two external masters. This makes me sad.
,
Dec 9 2016
,
Dec 9 2016
,
Dec 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/3439f40e09fc8ca5291e21a22fc22d05a947b9dd commit 3439f40e09fc8ca5291e21a22fc22d05a947b9dd Author: Sean McCullough <seanmccullough@chromium.org> Date: Fri Dec 09 23:00:56 2016 [som] Show internal master restarts on /trooper page Also refactores some gitiles access stuff to add easy caching. BUG= 672265 Change-Id: Id4eed70fe646d8bf1f11c87da9c5a8bd24c597e4 Reviewed-on: https://chromium-review.googlesource.com/418475 Commit-Queue: Sean McCullough <seanmccullough@chromium.org> Reviewed-by: Tiffany Zhang <zhangtiff@chromium.org> [modify] https://crrev.com/3439f40e09fc8ca5291e21a22fc22d05a947b9dd/go/src/infra/appengine/sheriff-o-matic/som/alerts.go [modify] https://crrev.com/3439f40e09fc8ca5291e21a22fc22d05a947b9dd/go/src/infra/appengine/sheriff-o-matic/som/gitiles.go [modify] https://crrev.com/3439f40e09fc8ca5291e21a22fc22d05a947b9dd/go/src/infra/appengine/sheriff-o-matic/som/pubsub.go
,
Dec 9 2016
Fix should go out with the next release |
||||
►
Sign in to add a comment |
||||
Comment 1 by zhangtiff@chromium.org
, Dec 7 2016