node: memory-mgr: Update log levels and add logs to capture state update

swatisehgal · swatisehgal · commit 07d83acce560 · 2025-02-06T16:41:02.000Z
Signed-off-by: Swati Sehgal &lt;swsehgal@redhat.com&gt;
diff --git a/pkg/kubelet/cm/memorymanager/memory_manager.go b/pkg/kubelet/cm/memorymanager/memory_manager.go
@@ -344,15 +344,15 @@ func (m *manager) removeStaleState() {
 	for podUID := range assignments {
 		for containerName := range assignments[podUID] {
 			if _, ok := activeContainers[podUID][containerName]; !ok {
-				klog.InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)
+				klog.V(2).InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)
 				m.policyRemoveContainerByRef(podUID, containerName)
 			}
 		}
 	}
 
 	m.containerMap.Visit(func(podUID, containerName, containerID string) {
 		if _, ok := activeContainers[podUID][containerName]; !ok {
-			klog.InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)
+			klog.V(2).InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)
 			m.policyRemoveContainerByRef(podUID, containerName)
 		}
 	})
diff --git a/pkg/kubelet/cm/memorymanager/state/state_mem.go b/pkg/kubelet/cm/memorymanager/state/state_mem.go
@@ -94,6 +94,7 @@ func (s *stateMemory) SetMemoryAssignments(assignments ContainerMemoryAssignment
 	defer s.Unlock()
 
 	s.assignments = assignments.Clone()
+	klog.V(5).InfoS("Updated Memory assignments", "assignments", assignments)
 }
 
 // Delete deletes corresponding Blocks from ContainerMemoryAssignments

Original file line number	Diff line number	Diff line change
`@@ -344,15 +344,15 @@ func (m *manager) removeStaleState() {`
`344`	`344`	`for podUID := range assignments {`
`345`	`345`	`for containerName := range assignments[podUID] {`
`346`	`346`	`if _, ok := activeContainers[podUID][containerName]; !ok {`
`347`		`- klog.InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)`
	`347`	`+ klog.V(2).InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)`
`348`	`348`	`m.policyRemoveContainerByRef(podUID, containerName)`
`349`	`349`	`}`
`350`	`350`	`}`
`351`	`351`	`}`
`352`	`352`
`353`	`353`	`m.containerMap.Visit(func(podUID, containerName, containerID string) {`
`354`	`354`	`if _, ok := activeContainers[podUID][containerName]; !ok {`
`355`		`- klog.InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)`
	`355`	`+ klog.V(2).InfoS("RemoveStaleState removing state", "podUID", podUID, "containerName", containerName)`
`356`	`356`	`m.policyRemoveContainerByRef(podUID, containerName)`
`357`	`357`	`}`
`358`	`358`	`})`
Original file line number	Diff line number	Diff line change
`@@ -94,6 +94,7 @@ func (s *stateMemory) SetMemoryAssignments(assignments ContainerMemoryAssignment`
`94`	`94`	`defer s.Unlock()`
`95`	`95`
`96`	`96`	`s.assignments = assignments.Clone()`
	`97`	`+ klog.V(5).InfoS("Updated Memory assignments", "assignments", assignments)`
`97`	`98`	`}`
`98`	`99`
`99`	`100`	`// Delete deletes corresponding Blocks from ContainerMemoryAssignments`