From 966d30470488e4b4e751386ef1bb38c8dd33ce40 Mon Sep 17 00:00:00 2001 From: Harshal Patil Date: Tue, 11 Jun 2024 16:42:59 -0400 Subject: [PATCH] Report correct error after validating the root container Signed-off-by: Harshal Patil --- pkg/kubelet/cm/qos_container_manager_linux.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pkg/kubelet/cm/qos_container_manager_linux.go b/pkg/kubelet/cm/qos_container_manager_linux.go index abf4487ee5d..a740acbbb49 100644 --- a/pkg/kubelet/cm/qos_container_manager_linux.go +++ b/pkg/kubelet/cm/qos_container_manager_linux.go @@ -82,8 +82,9 @@ func (m *qosContainerManagerImpl) GetQOSContainersInfo() QOSContainersInfo { func (m *qosContainerManagerImpl) Start(getNodeAllocatable func() v1.ResourceList, activePods ActivePodsFunc) error { cm := m.cgroupManager rootContainer := m.cgroupRoot - if !cm.Exists(rootContainer) { - return fmt.Errorf("root container %v doesn't exist", rootContainer) + + if err := cm.Validate(rootContainer); err != nil { + return fmt.Errorf("error validating root container %v : %w", rootContainer, err) } // Top level for Qos containers are created only for Burstable