review updates

Signed-off-by: Jordan Keister <[email protected]>

Author: grokspawn

api/v1alpha1/clusterextension_types.go

@@ -97,7 +97,7 @@ type SourceConfig struct {
 	// catalog is used to configure how information is sourced from a catalog.
 	// This field is required when sourceType is "Catalog", and forbidden otherwise.
 	//
-	// +optional.
+	// +optional
 	Catalog *CatalogSource `json:"catalog,omitempty"`
 }
 
@@ -107,7 +107,7 @@ type SourceConfig struct {
 // +union
 type ClusterExtensionInstallConfig struct {
 	// namespace designates the kubernetes Namespace where bundle content
-	// for the 'packageName' package field will be applied and the necessary
+	// for the package, referenced in the 'packageName' field, will be applied and the necessary
 	// service account can be found.
 	// The bundle may contain cluster-scoped resources or resources that are
 	// applied to other Namespaces. This Namespace is expected to exist.
@@ -264,7 +264,7 @@ type CatalogSource struct {
 	// Each channel in the list must follow the DNS subdomain standard
 	// as defined in [RFC 1123]. It must contain only lowercase alphanumeric characters,
 	// hyphens (-) or periods (.), start and end with an alphanumeric character,
-	// and be no longer than 253 characters.
+	// and be no longer than 253 characters. No more than 256 channels can be specified.
 	//
 	// When specified, it is used to constrain the set of installable bundles and
 	// the automated upgrade path. This constraint is an AND operation with the
@@ -394,7 +394,6 @@ type CRDUpgradeSafetyPreflightConfig struct {
 	// performing an upgrade operation.
 	//
 	// +kubebuilder:validation:Enum:="None";"Strict"
-	// +kubebuilder:default:=Strict
 	// +kubebuilder:validation:Required
 	Enforcement CRDUpgradeSafetyEnforcement `json:"enforcement"`
 }
@@ -434,10 +433,10 @@ type BundleMetadata struct {
 	Name string `json:"name"`
 
 	// version is a required field and is a reference to the version that this bundle represents
-	// version follows the semantic versioning standard as defined in https://semver.org/, but permits a leading 'v' character.
+	// version follows the semantic versioning standard as defined in https://semver.org/.
 	//
 	// +kubebuilder:validation:Required
-	// +kubebuilder:validation:XValidation:rule="self.matches(\"^v?([0-9]+)(\\\\.[0-9]+)?(\\\\.[0-9]+)?(-([-0-9A-Za-z]+(\\\\.[-0-9A-Za-z]+)*))?(\\\\+([-0-9A-Za-z]+(-\\\\.[-0-9A-Za-z]+)*))?\")",message="version name must foo"
+	// +kubebuilder:validation:XValidation:rule="self.matches(\"^([0-9]+)(\\\\.[0-9]+)?(\\\\.[0-9]+)?(-([-0-9A-Za-z]+(\\\\.[-0-9A-Za-z]+)*))?(\\\\+([-0-9A-Za-z]+(-\\\\.[-0-9A-Za-z]+)*))?\")",message="version must be well-formed semver"
 	Version string `json:"version"`
 }
 
@@ -451,8 +450,8 @@ type ClusterExtensionStatus struct {
 	//
 	// The Progressing condition represents whether or not the ClusterExtension is advancing towards a new state.
 	// When Progressing is True and the Reason is Succeeded, the ClusterExtension is making progress towards a new state.
+	// When Progressing is True and the Reason is Retrying, the ClusterExtension has encountered an error that could be resolved on subsequent reconciliation attempts.
 	// When Progressing is False and the Reason is Blocked, the ClusterExtension has encountered an error that requires manual intervention for recovery.
-	// When Progressing is False and the Reason is Retrying, the ClusterExtension has encountered an error that could be resolved on subsequent reconciliation attempts.
 	//
 	// When the ClusterExtension is sourced from a catalog, if may also communicate a deprecation condition.
 	// These are indications from a package owner to guide users away from a particular package, channel, or bundle.
@@ -517,6 +516,8 @@ type ClusterExtensionList struct {
 	// +optional
 	metav1.ListMeta `json:"metadata,omitempty"`
 
+	// items is a required list of ClusterExtension objects.
+	//
 	// +kubebuilder:validation:Required
 	Items []ClusterExtension `json:"items"`
 }

config/base/crd/bases/olm.operatorframework.io_clusterextensions.yaml

@@ -71,7 +71,7 @@ spec:
                   namespace:
                     description: |-
                       namespace designates the kubernetes Namespace where bundle content
-                      for the 'packageName' package field will be applied and the necessary
+                      for the package, referenced in the 'packageName' field, will be applied and the necessary
                       service account can be found.
                       The bundle may contain cluster-scoped resources or resources that are
                       applied to other Namespaces. This Namespace is expected to exist.
@@ -105,7 +105,6 @@ spec:
                           consequences of upgrading a CRD, such as data loss.
                         properties:
                           enforcement:
-                            default: Strict
                             description: |-
                               enforcement is a required field, used to configure the state of the CRD Upgrade Safety pre-flight check.
 
@@ -211,7 +210,7 @@ spec:
                           Each channel in the list must follow the DNS subdomain standard
                           as defined in [RFC 1123]. It must contain only lowercase alphanumeric characters,
                           hyphens (-) or periods (.), start and end with an alphanumeric character,
-                          and be no longer than 253 characters.
+                          and be no longer than 253 characters. No more than 256 channels can be specified.
 
                           When specified, it is used to constrain the set of installable bundles and
                           the automated upgrade path. This constraint is an AND operation with the
@@ -477,8 +476,8 @@ spec:
 
                   The Progressing condition represents whether or not the ClusterExtension is advancing towards a new state.
                   When Progressing is True and the Reason is Succeeded, the ClusterExtension is making progress towards a new state.
+                  When Progressing is True and the Reason is Retrying, the ClusterExtension has encountered an error that could be resolved on subsequent reconciliation attempts.
                   When Progressing is False and the Reason is Blocked, the ClusterExtension has encountered an error that requires manual intervention for recovery.
-                  When Progressing is False and the Reason is Retrying, the ClusterExtension has encountered an error that could be resolved on subsequent reconciliation attempts.
 
                   When the ClusterExtension is sourced from a catalog, if may also communicate a deprecation condition.
                   These are indications from a package owner to guide users away from a particular package, channel, or bundle.
@@ -568,11 +567,11 @@ spec:
                       version:
                         description: |-
                           version is a required field and is a reference to the version that this bundle represents
-                          version follows the semantic versioning standard as defined in https://semver.org/, but permits a leading 'v' character.
+                          version follows the semantic versioning standard as defined in https://semver.org/.
                         type: string
                         x-kubernetes-validations:
-                        - message: version name must foo
-                          rule: self.matches("^v?([0-9]+)(\\.[0-9]+)?(\\.[0-9]+)?(-([-0-9A-Za-z]+(\\.[-0-9A-Za-z]+)*))?(\\+([-0-9A-Za-z]+(-\\.[-0-9A-Za-z]+)*))?")
+                        - message: version must be well-formed semver
+                          rule: self.matches("^([0-9]+)(\\.[0-9]+)?(\\.[0-9]+)?(-([-0-9A-Za-z]+(\\.[-0-9A-Za-z]+)*))?(\\+([-0-9A-Za-z]+(-\\.[-0-9A-Za-z]+)*))?")
                     required:
                     - name
                     - version