fixed panic when target object does not exist

raffaelespazzoli · raffaelespazzoli · commit fd2200303595 · 2022-07-25T11:15:43.000-04:00
Signed-off-by: raffaelespazzoli &lt;raffaele.spazzoli@gmail.com&gt;
diff --git a/pkg/util/lockedresourcecontroller/patch-reconciler.go b/pkg/util/lockedresourcecontroller/patch-reconciler.go
@@ -452,7 +452,7 @@ func (lpr *LockedPatchReconciler) Reconcile(ctx context.Context, request reconci
 	for i := range lpr.patch.SourceObjectRefs {
 		sourceObj, err := lpr.patch.SourceObjectRefs[i].GetReferencedObject(ctx, targetObj)
 		if err != nil {
-			lpr.log.Error(err, "unable to retrieve", "source", sourceObj)
+			lpr.log.Error(err, "unable to retrieve", "sourceObjectRef", lpr.patch.SourceObjectRefs[i])
 			return lpr.manageError(targetObj, err)
 		}
 		sourceMap, err := getSubMapFromObject(ctx, sourceObj, lpr.patch.SourceObjectRefs[i].FieldPath)

Original file line number	Diff line number	Diff line change
`@@ -452,7 +452,7 @@ func (lpr *LockedPatchReconciler) Reconcile(ctx context.Context, request reconci`
`452`	`452`	`for i := range lpr.patch.SourceObjectRefs {`
`453`	`453`	`sourceObj, err := lpr.patch.SourceObjectRefs[i].GetReferencedObject(ctx, targetObj)`
`454`	`454`	`if err != nil {`
`455`		`- lpr.log.Error(err, "unable to retrieve", "source", sourceObj)`
	`455`	`+ lpr.log.Error(err, "unable to retrieve", "sourceObjectRef", lpr.patch.SourceObjectRefs[i])`
`456`	`456`	`return lpr.manageError(targetObj, err)`
`457`	`457`	`}`
`458`	`458`	`sourceMap, err := getSubMapFromObject(ctx, sourceObj, lpr.patch.SourceObjectRefs[i].FieldPath)`