avoid plan cache when handling jobs (#142)

dweomer · web-flow · commit 50d5e4fa6cd5 · 2021-09-12T22:44:42.000-07:00
Attempting to address #134 Signed-off-by: Jacob Blain Christen <jacob@rancher.com>
diff --git a/pkg/upgrade/handle_batch.go b/pkg/upgrade/handle_batch.go
@@ -47,7 +47,7 @@ func (ctl *Controller) handleJobs(ctx context.Context) error {
 			return obj, deleteJob(jobs, obj, metav1.DeletePropagationBackground)
 		}
 		// get the plan being applied
-		plan, err := plans.Cache().Get(obj.Namespace, planName)
+		plan, err := plans.Get(obj.Namespace, planName, metav1.GetOptions{})
 		switch {
 		case errors.IsNotFound(err):
 			// plan is gone, delete

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ func (ctl *Controller) handleJobs(ctx context.Context) error {`
`47`	`47`	`return obj, deleteJob(jobs, obj, metav1.DeletePropagationBackground)`
`48`	`48`	`}`
`49`	`49`	`// get the plan being applied`
`50`		`- plan, err := plans.Cache().Get(obj.Namespace, planName)`
	`50`	`+ plan, err := plans.Get(obj.Namespace, planName, metav1.GetOptions{})`
`51`	`51`	`switch {`
`52`	`52`	`case errors.IsNotFound(err):`
`53`	`53`	`// plan is gone, delete`