Exclude already completed pods from being included in the overhead ca…

…lculation (#19)

* Excludes already successful pods from being included in the overhead calculation

* Fixes import alias issue

* Removes corev1 import alias

* Excludes failed pods as well from the overhead calculation

* Updates comment line to take into account recent changes

internal/extender/overhead.go

@@ -108,8 +108,8 @@ func (o *OverheadComputer) compute(ctx context.Context) {
 		if podsWithRRs[p.Name] {
 			continue
 		}
-		if p.Spec.NodeName == "" {
-			// pending pod
+		if p.Spec.NodeName == "" || p.Status.Phase == v1.PodSucceeded || p.Status.Phase == v1.PodFailed {
+			// pending pod or pod succeeded or failed
 			continue
 		}
 		node, err := o.nodeLister.Get(p.Spec.NodeName)