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 <adc@redhat.com>
redhat-openshift-ecosystem · May 15, 2024 · 8e57adc · 8e57adc
1 parent 83ee11a
commit 8e57adc
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 4 deletions.
diff --git a/internal/policy/operator/deployable_by_olm.go b/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,11 +251,19 @@ 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
+	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 = "preflight-" + packageName
+	}
+
 	return &operatorData{
 		CatalogImage:     catalogImage,
 		Channel:          channel,
 		PackageName:      packageName,
-		App:              packageName,
+		App:              appName,
 		InstallNamespace: packageName,
 		TargetNamespace:  packageName + "-target",
 		InstallModes:     installModes,

diff --git a/internal/policy/operator/deployable_by_olm_test.go b/internal/policy/operator/deployable_by_olm_test.go
@@ -108,7 +108,7 @@ var _ = Describe("DeployableByOLMCheck", func() {
 			BeforeEach(func() {
 				badSub := sub
 				Expect(deployableByOLMCheck.client.Get(testcontext, crclient.ObjectKey{
-					Name:      "testPackage",
+					Name:      "preflight-testPackage",
 					Namespace: "testPackage",
 				}, &badSub)).To(Succeed())
 				badSub.Status.InstalledCSV = ""

diff --git a/internal/policy/operator/operator_suite_test.go b/internal/policy/operator/operator_suite_test.go
@@ -152,7 +152,7 @@ var secret = corev1.Secret{
 
 var sub = operatorsv1alpha1.Subscription{
 	ObjectMeta: metav1.ObjectMeta{
-		Name:      "testPackage",
+		Name:      "preflight-testPackage",
 		Namespace: "testPackage",
 	},
 	Status: operatorsv1alpha1.SubscriptionStatus{
@@ -162,7 +162,7 @@ var sub = operatorsv1alpha1.Subscription{
 
 var og = operatorsv1.OperatorGroup{
 	ObjectMeta: metav1.ObjectMeta{
-		Name:      "testPackage",
+		Name:      "preflight-testPackage",
 		Namespace: "testPackage",
 	},
 	Status: operatorsv1.OperatorGroupStatus{