controller/registry: don't set memory limits for target

It's not allowed for core components to have memory limits, to ensure
that that source of crash-looping pods does not occur to the core
payload.

Signed-off-by: Steve Kuznetsov <[email protected]>

Author: stevekuznetsov

pkg/controller/registry/reconciler/reconciler.go

@@ -227,9 +227,6 @@ func Pod(source *operatorsv1alpha1.CatalogSource, name string, img string, saNam
 			if pod.Spec.Containers[0].Resources.Limits == nil {
 				pod.Spec.Containers[0].Resources.Limits = map[corev1.ResourceName]resource.Quantity{}
 			}
-			double := *grpcPodConfig.MemoryTarget
-			double.Add(double.DeepCopy())
-			pod.Spec.Containers[0].Resources.Limits[corev1.ResourceMemory] = double
 
 			grpcPodConfig.MemoryTarget.Format = resource.BinarySI
 			pod.Spec.Containers[0].Env = append(pod.Spec.Containers[0].Env, corev1.EnvVar{

pkg/controller/registry/reconciler/reconciler_test.go

@@ -150,9 +150,6 @@ func TestPodMemoryTarget(t *testing.T) {
 									corev1.ResourceCPU:    resource.MustParse("10m"),
 									corev1.ResourceMemory: resource.MustParse("5Mi"),
 								},
-								Limits: corev1.ResourceList{
-									corev1.ResourceMemory: resource.MustParse("10Mi"),
-								},
 							},
 							SecurityContext: &corev1.SecurityContext{
 								ReadOnlyRootFilesystem: pointer.Bool(false),