fix ttl_cache.removeExpiredLoop: remove Tick usage

Signed-off-by: Ivan.Makeev <[email protected]>
deckhouse · Dec 4, 2024 · 631509e · 631509e
1 parent 0f7e386
commit 631509e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/images/storage-network-controller/src/main.go b/images/storage-network-controller/src/main.go
@@ -110,6 +110,7 @@ func main() {
 		err := discoverer.DiscoveryLoop(ctx, *cfg, mgr)
 		if err != nil {
 			log.Error(err, "failed to discovery node")
+			cancel()
 			os.Exit(2)
 		}
 	}

diff --git a/images/storage-network-controller/src/pkg/cache/ttl_cache.go b/images/storage-network-controller/src/pkg/cache/ttl_cache.go
@@ -55,7 +55,8 @@ func NewTTL[K comparable, V any](ctx context.Context) *TTLCache[K, V] {
 
 // Iterate over the cache items every 5 sec and delete expired ones.
 func (c *TTLCache[K, V]) removeExpiredLoop(ctx context.Context) {
-	for range time.Tick(5 * time.Second) {
+	// endless loop
+	for {
 		select {
 		case <-ctx.Done():
 			// do nothing in case of cancel, just return from this gorouting
@@ -72,6 +73,9 @@ func (c *TTLCache[K, V]) removeExpiredLoop(ctx context.Context) {
 			}
 
 			c.mu.Unlock()
+
+			// sleep for 5 sec
+			time.Sleep(5 * time.Second)
 		}
 	}
 }