Merge pull request #948 from cheney-lin/fix/get_headroom

luomingmeng · web-flow · commit 546736a520e5 · 2025-08-26T19:00:44.000+08:00
fix(sysadvisor): pending Pods is considered as Active
diff --git a/pkg/util/native/pods.go b/pkg/util/native/pods.go
@@ -171,7 +171,7 @@ func PodIsReady(pod *v1.Pod) bool {
 
 // PodIsActive returns whether the pod is not terminated.
 func PodIsActive(pod *v1.Pod) bool {
-	return !PodIsTerminated(pod)
+	return !PodIsTerminated(pod) && !PodIsPending(pod)
 }
 
 // PodIsPending returns whether the pod is pending.

Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,7 @@ func PodIsReady(pod *v1.Pod) bool {`
`171`	`171`
`172`	`172`	`// PodIsActive returns whether the pod is not terminated.`
`173`	`173`	`func PodIsActive(pod *v1.Pod) bool {`
`174`		`- return !PodIsTerminated(pod)`
	`174`	`+ return !PodIsTerminated(pod) && !PodIsPending(pod)`
`175`	`175`	`}`
`176`	`176`
`177`	`177`	`// PodIsPending returns whether the pod is pending.`