label length limitation (#341)

* label length cutoff

Signed-off-by: Allen Li <[email protected]>

* update character type

Signed-off-by: Allen Li <[email protected]>

* remove unnecessary conversion

Signed-off-by: Allen Li <[email protected]>

* remove string conversion

Signed-off-by: Allen Li <[email protected]>

---------

Signed-off-by: Allen Li <[email protected]>

Author: qpdpQ

controllers/common/util.go

@@ -156,3 +156,10 @@ func GetResourcesDynamically(ctx context.Context, dynamic dynamic.Interface, gro
 
 	return list.Items, nil
 }
+
+func GetFirstNCharacter(str string, n int) string {
+	if n >= len(str) {
+		return str
+	}
+	return str[:n]
+}

controllers/namespacescope_controller.go

@@ -1015,9 +1015,11 @@ func (r *NamespaceScopeReconciler) CSVReconcile(ctx context.Context, req ctrl.Re
 	for _, packageName := range candidateSet.ToSlice() {
 		managedCSVList = append(managedCSVList, packageName.(string))
 		csvList := &olmv1alpha1.ClusterServiceVersionList{}
+		labelKey := packageName.(string) + "." + instance.Namespace
+		labelKey = util.GetFirstNCharacter(labelKey, 63)
 		if err := r.Client.List(ctx, csvList, &client.ListOptions{
 			LabelSelector: labels.SelectorFromSet(map[string]string{
-				"operators.coreos.com/" + packageName.(string) + "." + instance.Namespace: "",
+				"operators.coreos.com/" + labelKey: "",
 			})}); err != nil {
 			klog.Error(err)
 			return ctrl.Result{}, err