review feedback

Author: razvan

deploy/helm/spark-k8s-operator/crds/crds.yaml

@@ -1119,7 +1119,10 @@ spec:
               properties:
                 clusterConfig:
                   default: {}
-                  description: Global Spark history server configuration that applies to all roles and role groups. Currently not in use.
+                  description: |-
+                    Global Spark history server configuration that applies to all roles.
+
+                    This was previously used to hold the listener configuration, which has since moved to the role configuration.
                   type: object
                 image:
                   anyOf:
@@ -1867,7 +1870,10 @@ spec:
                   type: array
                 clusterConfig:
                   default: {}
-                  description: Global Spark Connect server configuration that applies to all roles. Currently not in use.
+                  description: |-
+                    Global Spark Connect server configuration that applies to all roles.
+
+                    This was previously used to hold the listener configuration, which has since moved to the server configuration.
                   type: object
                 clusterOperation:
                   default:

rust/operator-binary/src/connect/crd.rs

@@ -79,7 +79,9 @@ pub mod versioned {
         pub image: ProductImage,
 
         /// Global Spark Connect server configuration that applies to all roles.
-        /// Currently not in use.
+        ///
+        /// This was previously used to hold the listener configuration, which has since moved
+        /// to the server configuration.
         #[serde(default)]
         pub cluster_config: v1alpha1::SparkConnectServerClusterConfig,
 

rust/operator-binary/src/crd/history.rs

@@ -81,8 +81,10 @@ pub mod versioned {
     pub struct SparkHistoryServerSpec {
         pub image: ProductImage,
 
-        /// Global Spark history server configuration that applies to all roles and role groups.
-        /// Currently not in use.
+        /// Global Spark history server configuration that applies to all roles.
+        ///
+        /// This was previously used to hold the listener configuration, which has since moved
+        /// to the role configuration.
         #[serde(default)]
         pub cluster_config: v1alpha1::SparkHistoryServerClusterConfig,