update: change ossm sidecar as labels on authornio deployment (#1702)

* update: add ossm sidecar as labels on authornio deployment - as a soft handover, to keep annotation for now, just add new one - we will need to remove the annotation one later as cleanup Signed-off-by: Wen Zhou <wenzhou@redhat.com> * update: remove annotations Signed-off-by: Wen Zhou <wenzhou@redhat.com> --------- Signed-off-by: Wen Zhou <wenzhou@redhat.com>
opendatahub-io · Feb 27, 2025 · 7eeb024 · 7eeb024
1 parent 2fa3afc
commit 7eeb024
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/controllers/dscinitialization/resources/authorino/deployment.injection.patch.tmpl.yaml b/controllers/dscinitialization/resources/authorino/deployment.injection.patch.tmpl.yaml
@@ -6,5 +6,5 @@ metadata:
 spec:
   template:
     metadata:
-      annotations:
+      labels:
         sidecar.istio.io/inject: "true"
diff --git a/controllers/dscinitialization/servicemesh_setup.go b/controllers/dscinitialization/servicemesh_setup.go
@@ -209,7 +209,7 @@ func (r *DSCInitializationReconciler) authorizationFeatures(instance *dsciv1.DSC
 
 			// We do not have the control over deployment resource creation.
 			// It is created by Authorino operator using Authorino CR and labels are not propagated from Authorino CR to spec.template
-			// See https://issues.redhat.com/browse/RHOAIENG-5494
+			// See https://issues.redhat.com/browse/RHOAIENG-5494 and https://github.com/Kuadrant/authorino-operator/pull/243
 			//
 			// To make it part of Service Mesh we have to patch it with injection
 			// enabled instead, otherwise it will not have proxy pod injected.