tree ad34f22c18cdb995c2a0209f3d2e0bf395913b11
parent 189b772438e5c966917b049f62d2e8cd1b4fbe66
author Kevin Lubick <kjlubick@google.com> 1619612423 -0400
committer Kevin Lubick <kjlubick@google.com> 1619616600 +0000

[gold] Make CLSummaries aware of triage activity

We now correctly identify stale cache entries as having to
do with seeing Changelist data AND triage actions taken
by a user.

We also warm the caches by taking into account the triage
actions.

This also changes how we handle stale cache entries. We try
to fetch the latest data, but if it's not available in time
(500ms), we return the stale data flagged as such (so the
client knows to retry it later).

We went with this short timeout approach to balance
keeping the RPC responsive but also serving fresh data now
if we can get it quickly.

Bug: skia:11917
Change-Id: I88c95ad6819fb81feeec914862fb79acf2b693be
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/401923
Reviewed-by: Leandro Lovisolo <lovisolo@google.com>