[CARRY] Remove RELATED_IMAGE_ from the handler image env var

This is causing a conflict with ART tooling.
bcrochet · Dec 4, 2020 · 849bd7d · 849bd7d
1 parent 5feee8e
commit 849bd7d
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/bundle/manifests/kubernetes-nmstate-operator.clusterserviceversion.yaml b/bundle/manifests/kubernetes-nmstate-operator.clusterserviceversion.yaml
@@ -122,7 +122,7 @@ spec:
                 - name: PROFILER_PORT
                   value: "6060"
                 - name: RUN_OPERATOR
-                - name: RELATED_IMAGE_HANDLER_IMAGE
+                - name: HANDLER_IMAGE
                   value: quay.io/nmstate/kubernetes-nmstate-handler:latest
                 - name: HANDLER_IMAGE_PULL_POLICY
                   value: Always

diff --git a/controllers/nmstate_controller.go b/controllers/nmstate_controller.go
@@ -132,7 +132,7 @@ func (r *NMStateReconciler) applyNamespace(instance *nmstatev1beta1.NMState) err
 func (r *NMStateReconciler) applyRBAC(instance *nmstatev1beta1.NMState) error {
 	data := render.MakeRenderData()
 	data.Data["HandlerNamespace"] = os.Getenv("HANDLER_NAMESPACE")
-	data.Data["HandlerImage"] = os.Getenv("RELATED_IMAGE_HANDLER_IMAGE")
+	data.Data["HandlerImage"] = os.Getenv("HANDLER_IMAGE")
 	data.Data["HandlerPullPolicy"] = os.Getenv("HANDLER_IMAGE_PULL_POLICY")
 	data.Data["HandlerPrefix"] = os.Getenv("HANDLER_PREFIX")
 	return r.renderAndApply(instance, data, "rbac", true)
@@ -159,7 +159,7 @@ func (r *NMStateReconciler) applyHandler(instance *nmstatev1beta1.NMState) error
 	amd64AndCRNodeSelector["beta.kubernetes.io/arch"] = "amd64"
 
 	data.Data["HandlerNamespace"] = os.Getenv("HANDLER_NAMESPACE")
-	data.Data["HandlerImage"] = os.Getenv("RELATED_IMAGE_HANDLER_IMAGE")
+	data.Data["HandlerImage"] = os.Getenv("HANDLER_IMAGE")
 	data.Data["HandlerPullPolicy"] = os.Getenv("HANDLER_IMAGE_PULL_POLICY")
 	data.Data["HandlerPrefix"] = os.Getenv("HANDLER_PREFIX")
 	data.Data["WebhookNodeSelector"] = amd64ArchOnMasterNodeSelector

diff --git a/controllers/nmstate_controller_test.go b/controllers/nmstate_controller_test.go
@@ -62,7 +62,7 @@ var _ = Describe("NMState controller reconcile", func() {
 		reconciler.Scheme = s
 		reconciler.Log = ctrl.Log.WithName("controllers").WithName("NMState")
 		os.Setenv("HANDLER_NAMESPACE", handlerNamespace)
-		os.Setenv("RELATED_IMAGE_HANDLER_IMAGE", handlerImage)
+		os.Setenv("HANDLER_IMAGE", handlerImage)
 		os.Setenv("HANDLER_IMAGE_PULL_POLICY", imagePullPolicy)
 		os.Setenv("HANDLER_PREFIX", handlerPrefix)
 	})

diff --git a/deploy/operator/operator.yaml b/deploy/operator/operator.yaml
@@ -44,7 +44,7 @@ spec:
               value: "6060"
             - name: RUN_OPERATOR
               value: ""
-            - name: RELATED_IMAGE_HANDLER_IMAGE
+            - name: HANDLER_IMAGE
               value: {{ .HandlerImage }}
             - name: HANDLER_IMAGE_PULL_POLICY
               value: {{ .HandlerPullPolicy }}

diff --git a/manifests/4.7/kubernetes-nmstate-operator.v4.7.0.clusterserviceversion.yaml b/manifests/4.7/kubernetes-nmstate-operator.v4.7.0.clusterserviceversion.yaml
@@ -128,7 +128,7 @@ spec:
                 - name: PROFILER_PORT
                   value: "6060"
                 - name: RUN_OPERATOR
-                - name: RELATED_IMAGE_HANDLER_IMAGE
+                - name: HANDLER_IMAGE
                   value: quay.io/openshift/origin-kubernetes-nmstate-handler:4.7
                 - name: HANDLER_IMAGE_PULL_POLICY
                   value: Always