Add last modified time to bug queue |
|||||
Issue descriptionI think for troopers for example, the last modified time for a bug is very useful to get an idea of how long it's been since they ACK'd something. Currently, it looks like the Monorail API response for issue lists doesn't provide this information, so we'll have to do some work to get it.
,
Feb 10 2017
,
May 24 2017
UI can be something like https://screenshot.googleplex.com/5wy9d20KjR6 The relevant code to modify is probably here: https://cs.chromium.org/chromium/infra/go/src/infra/appengine/sheriff-o-matic/som/bugqueue.go?l=91 +renjietang, this might be a good starter bug. I'll dig into the API to see if we can get last modified times in a set of Issue Detail responses, if not in the Issue List response itself (latter would be preferable).
,
May 24 2017
Yeah, it looks like "updated" is just missing from the protobuf file we generate to Go stubs from. API doc: https://chromium.googlesource.com/infra/infra/+/master/appengine/monorail/doc/api.md#monorail_issues_list monorail RPC proto def: https://chromium.googlesource.com/infra/infra/+/master/go/src/infra/monorail/monorail.proto#26 Probably can just add a field to message Issue: "string updated = 19" or some such. And viola we have last updated timestamp in the issue list response.
,
May 24 2017
,
May 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/0ff96024a6420ef00d7d487bb9725d9b8aac09bd commit 0ff96024a6420ef00d7d487bb9725d9b8aac09bd Author: Renjie Tang <renjietang@google.com> Date: Thu May 25 18:42:54 2017 Added the time that the bug was last updated to be shown in the bug queue. Bug:671702 Change-Id: If0fdb0bfaf092ef9cf545fd9f5ee462220c039d5 Reviewed-on: https://chromium-review.googlesource.com/514433 Commit-Queue: Renjie Tang <renjietang@google.com> Reviewed-by: Sean McCullough <seanmccullough@chromium.org> [modify] https://crrev.com/0ff96024a6420ef00d7d487bb9725d9b8aac09bd/go/src/infra/monorail/monorail.pb.go [modify] https://crrev.com/0ff96024a6420ef00d7d487bb9725d9b8aac09bd/go/src/infra/monorail/monorail.infra_testing [modify] https://crrev.com/0ff96024a6420ef00d7d487bb9725d9b8aac09bd/go/src/infra/appengine/sheriff-o-matic/elements/som-bug-queue/som-bug-queue.html [modify] https://crrev.com/0ff96024a6420ef00d7d487bb9725d9b8aac09bd/go/src/infra/monorail/monorail.proto [modify] https://crrev.com/0ff96024a6420ef00d7d487bb9725d9b8aac09bd/go/src/infra/monorail/pb.discovery.go
,
Jun 29 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by zhangtiff@chromium.org
, Dec 6 2016