Fix: PVC used by a job doesn't get resize after the pod of the job completed

PhanLe1010 · PhanLe1010 · commit 251e56c0a40c · 2022-03-30T15:19:53.000-07:00
Once the job pod is terminated, remove the corresponding PVC from the
usedPVCs.

issue-175

Signed-off-by: Phan Le &lt;phan.le@suse.com&gt;
diff --git a/pkg/controller/controller.go b/pkg/controller/controller.go
@@ -161,7 +161,11 @@ func (ctrl *resizeController) updatePod(oldObj, newObj interface{}) {
 		return
 	}
 
-	ctrl.usedPVCs.addPod(pod)
+	if isPodTerminated(pod) {
+		ctrl.usedPVCs.removePod(pod)
+	} else {
+		ctrl.usedPVCs.addPod(pod)
+	}
 }
 
 func (ctrl *resizeController) updatePVC(oldObj, newObj interface{}) {

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,11 @@ func (ctrl *resizeController) updatePod(oldObj, newObj interface{}) {`
`161`	`161`	`return`
`162`	`162`	`}`
`163`	`163`
`164`		`- ctrl.usedPVCs.addPod(pod)`
	`164`	`+ if isPodTerminated(pod) {`
	`165`	`+ ctrl.usedPVCs.removePod(pod)`
	`166`	`+ } else {`
	`167`	`+ ctrl.usedPVCs.addPod(pod)`
	`168`	`+ }`
`165`	`169`	`}`
`166`	`170`
`167`	`171`	`func (ctrl *resizeController) updatePVC(oldObj, newObj interface{}) {`