From a9fc52269b8aab60248b6e4c5366216bc2191101 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Mon, 19 Jan 2026 09:33:00 +0100 Subject: 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. --- dashboard/app/cache.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'dashboard') 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 -- cgit mrf-deployment