Update OLM to use UID for OG Labels

Problem:
OLM is using an older version of the operator-framework/api repository
with which OLM applies a label with a blank value whose key is equal to
"olm.operatorgroup/.".
Kubernetes limits the lengths of label values and keys to 63 characters.
This limit can easily be overcome when the OperatorGroup has a long name
or when it is deployed in a namespace with a long name.

Solution:
Update OLM to use import the latest version of the operator-framework/api
repository that uses "olm.operatorgroup.uid/" as the key
for OperatorGroup labels. The length of this label will always be 58
characters as UIDs are 36 characters long.

Author: awgreene

go.mod

@@ -22,8 +22,8 @@ require (
 	github.com/onsi/gomega v1.9.0
 	github.com/openshift/api v0.0.0-20200331152225-585af27e34fd
 	github.com/openshift/client-go v0.0.0-20200326155132-2a6cd50aedd0
-	github.com/operator-framework/api v0.3.1
-	github.com/operator-framework/operator-registry v1.11.0
+	github.com/operator-framework/api v0.3.2
+	github.com/operator-framework/operator-registry v1.12.1
 	github.com/otiai10/copy v1.0.2
 	github.com/pkg/errors v0.9.1
 	github.com/prometheus/client_golang v1.2.1
@@ -36,19 +36,18 @@ require (
 	google.golang.org/grpc v1.27.0
 	gopkg.in/yaml.v2 v2.2.8
 	helm.sh/helm/v3 v3.1.2
-	k8s.io/api v0.18.0
-	k8s.io/apiextensions-apiserver v0.18.0
-	k8s.io/apimachinery v0.18.0
-	k8s.io/apiserver v0.18.0
-	k8s.io/client-go v0.18.0
-	k8s.io/code-generator v0.18.0
-	k8s.io/component-base v0.18.0
+	k8s.io/api v0.18.2
+	k8s.io/apiextensions-apiserver v0.18.2
+	k8s.io/apimachinery v0.18.2
+	k8s.io/apiserver v0.18.2
+	k8s.io/client-go v0.18.2
+	k8s.io/code-generator v0.18.2
+	k8s.io/component-base v0.18.2
 	k8s.io/klog v1.0.0
 	k8s.io/kube-aggregator v0.18.0
 	k8s.io/kube-openapi v0.0.0-20200121204235-bf4fb3bd569c
-	rsc.io/letsencrypt v0.0.3 // indirect
-	sigs.k8s.io/controller-runtime v0.5.2
-	sigs.k8s.io/controller-tools v0.2.8
+	sigs.k8s.io/controller-runtime v0.6.0
+	sigs.k8s.io/controller-tools v0.3.0
 	sigs.k8s.io/kind v0.7.0
 )
 
@@ -58,7 +57,6 @@ replace (
 	github.com/openshift/api => github.com/openshift/api v0.0.0-20200331152225-585af27e34fd // release-4.5
 	github.com/openshift/client-go => github.com/openshift/client-go v0.0.0-20200326155132-2a6cd50aedd0 // release-4.5
 
-	github.com/operator-framework/api => github.com/operator-framework/api v0.3.1
 	github.com/prometheus/client_golang => github.com/prometheus/client_golang v0.9.3-0.20190127221311-3c4408c8b829
 	google.golang.org/grpc => google.golang.org/grpc v1.26.0 // https://github.com/etcd-io/etcd/issues/11563
 

go.sum

@@ -56,7 +56,10 @@ func (i *StrategyDeploymentInstaller) createOrUpdateWebhook(caPEM []byte, desc v
 	if err != nil || len(operatorGroups) != 1 {
 		return fmt.Errorf("Error retrieving OperatorGroup info")
 	}
-	ogNamespacelabelSelector := operatorGroups[0].NamespaceLabelSelector()
+	ogNamespacelabelSelector, err := operatorGroups[0].NamespaceLabelSelector()
+	if err != nil {
+		return err
+	}
 
 	switch desc.Type {
 	case v1alpha1.ValidatingAdmissionWebhook:

pkg/controller/install/webhook.go

@@ -776,7 +776,10 @@ func (a *Operator) syncNamespace(obj interface{}) error {
 			if namespace.Labels == nil {
 				namespace.Labels = make(map[string]string, 1)
 			}
-			ogLabelKey, ogLabelValue := group.OGLabelKeyAndValue()
+			ogLabelKey, ogLabelValue, err := group.OGLabelKeyAndValue()
+			if err != nil {
+				return err
+			}
 			namespace.Labels[ogLabelKey] = ogLabelValue
 		}
 	}

pkg/controller/operators/olm/operator.go

@@ -3337,7 +3337,7 @@ var _ = Describe("CSV", func() {
 		require.NoError(GinkgoT(), err)
 
 		expected := &metav1.LabelSelector{
-			MatchLabels:      map[string]string{"olm.operatorgroup/" + namespace.Name + "." + og.GetName(): ""},
+			MatchLabels:      map[string]string{fmt.Sprintf("olm.operatorgroup.uid/%s", og.GetUID()): ""},
 			MatchExpressions: []metav1.LabelSelectorRequirement(nil),
 		}
 

test/e2e/csv_e2e_test.go

@@ -2086,7 +2086,7 @@ var _ = Describe("Operator Group", func() {
 				TargetNamespaces: []string{},
 			},
 		}
-		_, err := crc.OperatorsV1().OperatorGroups(testNamespaceA).Create(context.TODO(), operatorGroup, metav1.CreateOptions{})
+		operatorGroup, err := crc.OperatorsV1().OperatorGroups(testNamespaceA).Create(context.TODO(), operatorGroup, metav1.CreateOptions{})
 		require.NoError(GinkgoT(), err)
 
 		// Cleanup OperatorGroup
@@ -2096,7 +2096,7 @@ var _ = Describe("Operator Group", func() {
 		}()
 
 		// Create the OperatorGroup Label
-		ogLabel := fmt.Sprintf("olm.operatorgroup/%s.%s", testNamespaceA, operatorGroup.GetName())
+		ogLabel := fmt.Sprintf("olm.operatorgroup.uid/%s", operatorGroup.GetUID())
 
 		// Create list options
 		listOptions := metav1.ListOptions{
@@ -2186,11 +2186,11 @@ var _ = Describe("Operator Group", func() {
 				TargetNamespaces: testNamespaces,
 			},
 		}
-		_, err := crc.OperatorsV1().OperatorGroups(testNamespaceA).Create(context.TODO(), operatorGroup, metav1.CreateOptions{})
+		operatorGroup, err := crc.OperatorsV1().OperatorGroups(testNamespaceA).Create(context.TODO(), operatorGroup, metav1.CreateOptions{})
 		require.NoError(GinkgoT(), err)
 
 		// Create the OperatorGroup Label
-		ogLabel := fmt.Sprintf("olm.operatorgroup/%s.%s", testNamespaceA, operatorGroup.GetName())
+		ogLabel := fmt.Sprintf("olm.operatorgroup.uid/%s", operatorGroup.GetUID())
 
 		// Create list options
 		listOptions := metav1.ListOptions{