From c75a3e717e20410a61f1736c1200771a06d22a74 Mon Sep 17 00:00:00 2001 From: Tim Allclair Date: Fri, 25 Oct 2024 13:32:36 -0700 Subject: [PATCH] More precise allocatedPod name usage --- pkg/kubelet/kubelet.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/kubelet/kubelet.go b/pkg/kubelet/kubelet.go index 316f5fbee69..75137b20bf3 100644 --- a/pkg/kubelet/kubelet.go +++ b/pkg/kubelet/kubelet.go @@ -2795,7 +2795,7 @@ func (kl *Kubelet) handlePodResourcesResize(pod *v1.Pod) (*v1.Pod, error) { allocatedPod, updated := kl.statusManager.UpdatePodFromAllocation(pod) if !updated { // Pod is not resizing, nothing more to do here. - return pod, nil + return allocatedPod, nil } kl.podResizeMutex.Lock() @@ -2810,9 +2810,9 @@ func (kl *Kubelet) handlePodResourcesResize(pod *v1.Pod) (*v1.Pod, error) { } if resizeStatus != "" { // Save resize decision to checkpoint - if err := kl.statusManager.SetPodResizeStatus(pod.UID, resizeStatus); err != nil { + if err := kl.statusManager.SetPodResizeStatus(allocatedPod.UID, resizeStatus); err != nil { //TODO(vinaykul,InPlacePodVerticalScaling): Can we recover from this in some way? Investigate - klog.ErrorS(err, "SetPodResizeStatus failed", "pod", klog.KObj(pod)) + klog.ErrorS(err, "SetPodResizeStatus failed", "pod", klog.KObj(allocatedPod)) } } return allocatedPod, nil