From c40f70cfb96704f395470ead20d6ad8e1ff62ea3 Mon Sep 17 00:00:00 2001 From: Daniel Franz Date: Thu, 9 Mar 2023 15:04:56 -0800 Subject: [PATCH] Registry Pod Controller Flag Sets the marketplace pod's ownerreference to have `controller: true` to remove need for forceful node drain. Signed-off-by: Daniel Franz --- pkg/controller/operators/catalog/operator_test.go | 2 +- pkg/controller/registry/reconciler/configmap.go | 2 +- pkg/controller/registry/reconciler/grpc.go | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/controller/operators/catalog/operator_test.go b/pkg/controller/operators/catalog/operator_test.go index 2c351d4a2e..dd5362a4b8 100644 --- a/pkg/controller/operators/catalog/operator_test.go +++ b/pkg/controller/operators/catalog/operator_test.go @@ -1808,7 +1808,7 @@ func toManifest(t *testing.T, obj runtime.Object) string { func pod(s v1alpha1.CatalogSource) *corev1.Pod { pod := reconciler.Pod(&s, "registry-server", s.Spec.Image, s.GetName(), s.GetLabels(), s.GetAnnotations(), 5, 10, 1001) - ownerutil.AddOwner(pod, &s, false, false) + ownerutil.AddOwner(pod, &s, false, true) return pod } diff --git a/pkg/controller/registry/reconciler/configmap.go b/pkg/controller/registry/reconciler/configmap.go index f264d1bee3..11a87a8746 100644 --- a/pkg/controller/registry/reconciler/configmap.go +++ b/pkg/controller/registry/reconciler/configmap.go @@ -105,7 +105,7 @@ func (s *configMapCatalogSourceDecorator) Pod(image string) *corev1.Pod { pod := Pod(s.CatalogSource, "configmap-registry-server", image, "", s.Labels(), s.Annotations(), 5, 5, s.runAsUser) pod.Spec.ServiceAccountName = s.GetName() + ConfigMapServerPostfix pod.Spec.Containers[0].Command = []string{"configmap-server", "-c", s.Spec.ConfigMap, "-n", s.GetNamespace()} - ownerutil.AddOwner(pod, s.CatalogSource, false, false) + ownerutil.AddOwner(pod, s.CatalogSource, false, true) return pod } diff --git a/pkg/controller/registry/reconciler/grpc.go b/pkg/controller/registry/reconciler/grpc.go index 93e28e6286..117c081b23 100644 --- a/pkg/controller/registry/reconciler/grpc.go +++ b/pkg/controller/registry/reconciler/grpc.go @@ -124,7 +124,7 @@ func (s *grpcCatalogSourceDecorator) ServiceAccount() *corev1.ServiceAccount { func (s *grpcCatalogSourceDecorator) Pod(saName string) *corev1.Pod { pod := Pod(s.CatalogSource, "registry-server", s.Spec.Image, saName, s.Labels(), s.Annotations(), 5, 10, s.createPodAsUser) - ownerutil.AddOwner(pod, s.CatalogSource, false, false) + ownerutil.AddOwner(pod, s.CatalogSource, false, true) return pod }