From 424c7ca7e5895f9fff169fd1b57f56264c039c60 Mon Sep 17 00:00:00 2001 From: Tim Allclair Date: Wed, 29 Jan 2025 11:59:35 -0800 Subject: [PATCH] Remove unused ClearState function --- pkg/kubelet/status/state/state.go | 1 - pkg/kubelet/status/state/state_checkpoint.go | 12 ------------ pkg/kubelet/status/state/state_mem.go | 10 ---------- 3 files changed, 23 deletions(-) diff --git a/pkg/kubelet/status/state/state.go b/pkg/kubelet/status/state/state.go index 10a66be46c3..ada73692448 100644 --- a/pkg/kubelet/status/state/state.go +++ b/pkg/kubelet/status/state/state.go @@ -50,7 +50,6 @@ type writer interface { SetPodResourceAllocation(PodResourceAllocation) error SetPodResizeStatus(podUID string, resizeStatus v1.PodResizeStatus) Delete(podUID string, containerName string) error - ClearState() error } // State interface provides methods for tracking and setting pod resource allocation diff --git a/pkg/kubelet/status/state/state_checkpoint.go b/pkg/kubelet/status/state/state_checkpoint.go index 2e1ca729100..c4c0291d02b 100644 --- a/pkg/kubelet/status/state/state_checkpoint.go +++ b/pkg/kubelet/status/state/state_checkpoint.go @@ -154,14 +154,6 @@ func (sc *stateCheckpoint) Delete(podUID string, containerName string) error { return sc.storeState() } -// ClearState clears the state and saves it in a checkpoint -func (sc *stateCheckpoint) ClearState() error { - sc.mux.Lock() - defer sc.mux.Unlock() - sc.cache.ClearState() - return sc.storeState() -} - type noopStateCheckpoint struct{} // NewNoopStateCheckpoint creates a dummy state checkpoint manager @@ -194,7 +186,3 @@ func (sc *noopStateCheckpoint) SetPodResizeStatus(_ string, _ v1.PodResizeStatus func (sc *noopStateCheckpoint) Delete(_ string, _ string) error { return nil } - -func (sc *noopStateCheckpoint) ClearState() error { - return nil -} diff --git a/pkg/kubelet/status/state/state_mem.go b/pkg/kubelet/status/state/state_mem.go index c8107dacbe2..0a0974de237 100644 --- a/pkg/kubelet/status/state/state_mem.go +++ b/pkg/kubelet/status/state/state_mem.go @@ -116,13 +116,3 @@ func (s *stateMemory) Delete(podUID string, containerName string) error { s.deleteContainer(podUID, containerName) return nil } - -func (s *stateMemory) ClearState() error { - s.Lock() - defer s.Unlock() - - s.podAllocation = make(PodResourceAllocation) - s.podResizeStatus = make(PodResizeStatus) - klog.V(3).InfoS("Cleared state") - return nil -}