Merge pull request #1000 from boblee0717/bob_cv1

luomingmeng · web-flow · commit 7ada70d557ad · 2025-10-20T15:13:27.000+08:00
fix NPE if the pod does not exist
diff --git a/pkg/agent/qrm-plugins/cpu/dynamicpolicy/policy_advisor_handler.go b/pkg/agent/qrm-plugins/cpu/dynamicpolicy/policy_advisor_handler.go
@@ -598,7 +598,7 @@ func (p *DynamicPolicy) checkAndApplyAllPodsQuota(calculationInfo *advisorsvc.Ca
 	for _, podDir := range podDirs {
 		pod, podRelativePath, err := p.getPodAndRelativePath(calculationInfo.CgroupPath, podDir, podsPathMap)
 		if err != nil {
-			general.Warningf("getPodAndRelativePath error for pod %s: %v", pod.Name, err)
+			general.Warningf("getPodAndRelativePath error for pod dir %s: %v", podDir, err)
 			continue
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -598,7 +598,7 @@ func (p DynamicPolicy) checkAndApplyAllPodsQuota(calculationInfo advisorsvc.Ca`
`598`	`598`	`for _, podDir := range podDirs {`
`599`	`599`	`pod, podRelativePath, err := p.getPodAndRelativePath(calculationInfo.CgroupPath, podDir, podsPathMap)`
`600`	`600`	`if err != nil {`
`601`		`- general.Warningf("getPodAndRelativePath error for pod %s: %v", pod.Name, err)`
	`601`	`+ general.Warningf("getPodAndRelativePath error for pod dir %s: %v", podDir, err)`
`602`	`602`	`continue`
`603`	`603`	`}`
`604`	`604`