diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2026-01-19 09:33:00 +0100 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2026-01-19 08:45:33 +0000 |
| commit | a9fc52269b8aab60248b6e4c5366216bc2191101 (patch) | |
| tree | 96e825734baf154395724e6f6a83edb6ecc67e84 /dashboard | |
| parent | 6ccb48f81a0242c3ba2ddd583ded0582364eeacd (diff) | |
dashboard/app: don't treat memcache.ErrNotStored as an error
We have a bunch of "failed to throttle: memcache: item not stored" errors in logs.
It seems that the cache item is evicted before we do CompareAndSwap.
Diffstat (limited to 'dashboard')
| -rw-r--r-- | dashboard/app/cache.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dashboard/app/cache.go b/dashboard/app/cache.go index 545279203..8d642c1b2 100644 --- a/dashboard/app/cache.go +++ b/dashboard/app/cache.go @@ -314,7 +314,7 @@ func ThrottleRequest(c context.Context, requesterID string) (bool, error) { item.Expiration = cfg.Window item.Object = obj err = memcache.Gob.CompareAndSwap(c, item) - if err == memcache.ErrCASConflict { + if err == memcache.ErrCASConflict || err == memcache.ErrNotStored { if ok { // Only retry if we approved the query. // If we denied and there was a concurrent write |