adding logic to deployablebyolm to validate that packagename complies…

… with DNS-1035 labeling, ensuring that CatalogSources can be created

Signed-off-by: Adam D. Cornett <[email protected]>

internal/policy/operator/deployable_by_olm.go

@@ -28,6 +28,7 @@ import (
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
 	apiruntime "k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
+	"k8s.io/apimachinery/pkg/util/validation"
 	ctrl "sigs.k8s.io/controller-runtime"
 	crclient "sigs.k8s.io/controller-runtime/pkg/client"
 )
@@ -214,6 +215,8 @@ func checkImageSource(ctx context.Context, operatorImages []string) bool {
 }
 
 func (p *DeployableByOlmCheck) operatorMetadata(ctx context.Context, bundleRef image.ImageReference) (*operatorData, error) {
+	logger := logr.FromContextOrDiscard(ctx)
+
 	// retrieve the operator metadata from bundle image
 	annotationsFileName := filepath.Join(bundleRef.ImageFSPath, "metadata", "annotations.yaml")
 	annotationsFile, err := os.Open(annotationsFileName)
@@ -248,13 +251,22 @@ func (p *DeployableByOlmCheck) operatorMetadata(ctx context.Context, bundleRef i
 		installModes[val.Type] = val
 	}
 
+	// validating that package name complies with DNS-1035 labeling constraints
+	// ensuring that CatalogSources can be created/referenced in cluster and reconciles properly
+	// if there is an error we need to prefix the name so the install/reconciles can continue
+	appName := packageName
+	if msgs := validation.IsDNS1035Label(packageName); len(msgs) != 0 {
+		logger.V(log.DBG).Info(fmt.Sprintf("package name %s does not comply with DNS-1035, prefixing to avoid errors", packageName))
+		appName = "p-" + packageName
+	}
+
 	return &operatorData{
 		CatalogImage:     catalogImage,
 		Channel:          channel,
 		PackageName:      packageName,
-		App:              packageName,
-		InstallNamespace: packageName,
-		TargetNamespace:  packageName + "-target",
+		App:              appName,
+		InstallNamespace: appName,
+		TargetNamespace:  appName + "-target",
 		InstallModes:     installModes,
 	}, nil
 }

internal/policy/operator/deployable_by_olm_test.go

@@ -72,7 +72,7 @@ var _ = Describe("DeployableByOLMCheck", func() {
 
 				// changing the namespace since OwnNamespace operators CSV get applied to `InstallNamespace`
 				ownCSV := csv.DeepCopy()
-				ownCSV.Namespace = "testPackage"
+				ownCSV.Namespace = "p-testPackage"
 
 				deployableByOLMCheck.client = clientBuilder.
 					WithObjects(ownCSV).
@@ -108,8 +108,8 @@ var _ = Describe("DeployableByOLMCheck", func() {
 			BeforeEach(func() {
 				badSub := sub
 				Expect(deployableByOLMCheck.client.Get(testcontext, crclient.ObjectKey{
-					Name:      "testPackage",
-					Namespace: "testPackage",
+					Name:      "p-testPackage",
+					Namespace: "p-testPackage",
 				}, &badSub)).To(Succeed())
 				badSub.Status.InstalledCSV = ""
 				Expect(deployableByOLMCheck.client.Update(testcontext, &badSub, &crclient.UpdateOptions{})).To(Succeed())

internal/policy/operator/operator_suite_test.go

@@ -105,7 +105,7 @@ var pods = corev1.PodList{
 var csv = operatorsv1alpha1.ClusterServiceVersion{
 	ObjectMeta: metav1.ObjectMeta{
 		Name:      "csv-v0.0.0",
-		Namespace: "testPackage-target",
+		Namespace: "p-testPackage-target",
 	},
 	Spec: operatorsv1alpha1.ClusterServiceVersionSpec{},
 	Status: operatorsv1alpha1.ClusterServiceVersionStatus{
@@ -152,8 +152,8 @@ var secret = corev1.Secret{
 
 var sub = operatorsv1alpha1.Subscription{
 	ObjectMeta: metav1.ObjectMeta{
-		Name:      "testPackage",
-		Namespace: "testPackage",
+		Name:      "p-testPackage",
+		Namespace: "p-testPackage",
 	},
 	Status: operatorsv1alpha1.SubscriptionStatus{
 		InstalledCSV: "csv-v0.0.0",
@@ -162,8 +162,8 @@ var sub = operatorsv1alpha1.Subscription{
 
 var og = operatorsv1.OperatorGroup{
 	ObjectMeta: metav1.ObjectMeta{
-		Name:      "testPackage",
-		Namespace: "testPackage",
+		Name:      "p-testPackage",
+		Namespace: "p-testPackage",
 	},
 	Status: operatorsv1.OperatorGroupStatus{
 		LastUpdated: nil,