Merge pull request #523 from ecordell/dont-annotate-immediately

fix(olm): don't annotate namespaces on operator creation

Author: openshift-merge-robot

pkg/controller/operators/olm/operator.go

@@ -86,11 +86,6 @@ func NewOperator(crClient versioned.Interface, opClient operatorclient.ClientInt
 		op.RegisterQueueInformer(queueInformer)
 	}
 
-	// annotate namespaces that ALM operator manages
-	if err := namespaceAnnotator.AnnotateNamespaces(namespaces); err != nil {
-		return nil, err
-	}
-
 	// set up RBAC informers
 	informerFactory := informers.NewSharedInformerFactory(opClient.KubernetesInterface(), wakeupInterval)
 	roleInformer := informerFactory.Rbac().V1().Roles()

test/e2e/e2e-values-shift.yaml

@@ -22,7 +22,7 @@ package:
     ref: quay.io/coreos/olm:local
     pullPolicy: Always
   service:
-    internalPort: 443
+    internalPort: 5443
   commandArgs: -test.coverprofile=/tmp/catalog-coverage.cov
 
 e2e:

test/e2e/e2e-values.yaml

@@ -24,7 +24,7 @@ package:
     ref: quay.io/coreos/olm:local
     pullPolicy: IfNotPresent
   service:
-    internalPort: 443
+    internalPort: 5443
   commandArgs: -test.coverprofile=/tmp/catalog-coverage.cov
 
 e2e: