📝 Note for the future.

coderanger · coderanger · commit ef42fbad992c · 2020-12-03T01:49:43.000-08:00
diff --git a/components/migrations.go b/components/migrations.go
@@ -301,6 +301,7 @@ func (_ *migrationsComponent) findOwners(ctx *cu.Context, obj cu.Object) ([]cu.O
 		}
 		err = ctx.Client.Get(ctx, types.NamespacedName{Name: ref.Name, Namespace: namespace}, ownerObj)
 		if err != nil {
+			// TODO IMPORTANT If this is a 403, don't bubble up the error. Probably a custom type we don't have access to, just pretend it's not there.
 			return nil, errors.Wrapf(err, "error finding object type for owner reference %v", ref)
 		}
 		obj = ownerObj.(cu.Object)

Original file line number	Diff line number	Diff line change
`@@ -301,6 +301,7 @@ func (_ migrationsComponent) findOwners(ctx cu.Context, obj cu.Object) ([]cu.O`
`301`	`301`	`}`
`302`	`302`	`err = ctx.Client.Get(ctx, types.NamespacedName{Name: ref.Name, Namespace: namespace}, ownerObj)`
`303`	`303`	`if err != nil {`
	`304`	`+ // TODO IMPORTANT If this is a 403, don't bubble up the error. Probably a custom type we don't have access to, just pretend it's not there.`
`304`	`305`	`return nil, errors.Wrapf(err, "error finding object type for owner reference %v", ref)`
`305`	`306`	`}`
`306`	`307`	`obj = ownerObj.(cu.Object)`