clarify maxOpenShiftVersion upgrade error message

Signed-off-by: Ankita Thomas <[email protected]>

Author: ankitathomas

Diff for: pkg/controller/operators/openshift/helpers.go

@@ -110,7 +110,7 @@ func (s skew) String() string {
 		return fmt.Sprintf("%s/%s has invalid %s properties: %s", s.namespace, s.name, MaxOpenShiftVersionProperty, s.err)
 	}
 
-	return fmt.Sprintf("%s/%s is incompatible with OpenShift versions greater than %s", s.namespace, s.name, s.maxOpenShiftVersion)
+	return fmt.Sprintf("%s/%s is incompatible with OpenShift minor versions greater than %s", s.namespace, s.name, s.maxOpenShiftVersion)
 }
 
 type transientError struct {
@@ -161,7 +161,8 @@ func incompatibleOperators(ctx context.Context, cli client.Client) (skews, error
 		if max == nil || max.GTE(*next) {
 			continue
 		}
-		s.maxOpenShiftVersion = max.String()
+
+		s.maxOpenShiftVersion = semver.Version{Major: max.Major, Minor: max.Minor}.String()
 
 		incompatible = append(incompatible, s)
 	}
@@ -234,7 +235,11 @@ func maxOpenShiftVersion(csv *operatorsv1alpha1.ClusterServiceVersion) (*semver.
 		return nil, fmt.Errorf(`Failed to parse "%s" as semver: %w`, value, err)
 	}
 
-	return &version, nil
+	truncatedVersion := semver.Version{Major: version.Major, Minor: version.Minor}
+	if !version.EQ(truncatedVersion) {
+		return nil, fmt.Errorf("property %s must specify only <major>.<minor> version, got invalid value %s", MaxOpenShiftVersionProperty, version)
+	}
+	return &truncatedVersion, nil
 }
 
 func notCopiedSelector() (labels.Selector, error) {

Diff for: pkg/controller/operators/openshift/helpers_test.go

@@ -304,12 +304,12 @@ func TestIncompatibleOperators(t *testing.T) {
 					{
 						name:                "beech",
 						namespace:           "default",
-						maxOpenShiftVersion: "1.0.0+build",
+						maxOpenShiftVersion: "1.0.0",
 					},
 					{
-						name:                "chestnut",
-						namespace:           "default",
-						maxOpenShiftVersion: "1.1.0-pre",
+						name:      "chestnut",
+						namespace: "default",
+						err:       fmt.Errorf("property olm.maxOpenShiftVersion must specify only <major>.<minor> version, got invalid value 1.1.0-pre"),
 					},
 					{
 						name:                "drupe",