Merge pull request #199 from PhanLe1010/release-1.2

k8s-ci-robot · web-flow · commit 90ef727bda9f · 2022-04-01T07:26:46.000-07:00
[Backoport][release-1.2] Fix: PVC used by a job doesn't get resize after the pod of the job completed
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{}) {`