Fix Rename ClusterRoles created by OperatorGroups issues.

Remove unused function.
Remove possibly verbose log statement.

Signed-off-by: Todd Short <[email protected]>
Upstream-repository: operator-lifecycle-manager
Upstream-commit: fa51d6f651791a0f6eb3e288140f5e495d4138ff

Author: tmshort

staging/operator-lifecycle-manager/pkg/controller/operators/olm/operatorgroup.go

@@ -79,8 +79,6 @@ func (a *Operator) syncOperatorGroups(obj interface{}) error {
 		"namespace":     op.GetNamespace(),
 	})
 
-	logger.Infof("syncing OperatorGroup %s/%s", op.GetNamespace(), op.GetName())
-
 	// Query OG in this namespace
 	groups, err := a.lister.OperatorsV1().OperatorGroupLister().OperatorGroups(op.GetNamespace()).List(labels.Everything())
 	if err != nil {

staging/operator-lifecycle-manager/pkg/lib/ownerutil/util.go

@@ -271,11 +271,6 @@ func CSVOwnerSelector(owner *operatorsv1alpha1.ClusterServiceVersion) labels.Sel
 	return labels.SelectorFromSet(OwnerLabel(owner, operatorsv1alpha1.ClusterServiceVersionKind))
 }
 
-// OperatorGroupOwnerSelector returns a label selector to find generated objects owned by owner
-func OperatorGroupOwnerSelector(owner *operatorsv1.OperatorGroup) labels.Selector {
-	return labels.SelectorFromSet(OwnerLabel(owner, operatorsv1.OperatorGroupKind))
-}
-
 // AddOwner adds an owner to the ownerref list.
 func AddOwner(object metav1.Object, owner Owner, blockOwnerDeletion, isController bool) {
 	// Most of the time we won't have TypeMeta on the object, so we infer it for types we know about