fix crd default, add comments

Author: maltesander

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

@@ -2332,14 +2332,14 @@ spec:
                       type: object
                       x-kubernetes-preserve-unknown-fields: true
                     role_config:
+                      default:
+                        listener_class: cluster-internal
                       properties:
                         listener_class:
                           default: cluster-internal
                           description: This field controls which [ListenerClass](https://docs.stackable.tech/home/nightly/listener-operator/listenerclass.html) is used to expose the Spark services.
                           type: string
                       type: object
-                  required:
-                    - role_config
                   type: object
                 vectorAggregatorConfigMapName:
                   description: Name of the Vector aggregator discovery ConfigMap. It must contain the key `ADDRESS` with the address of the Vector aggregator.

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

@@ -101,15 +101,17 @@ pub mod versioned {
         pub executor: Option<CommonConfiguration<ExecutorConfigFragment, JavaCommonConfig>>,
     }
 
-    /// This struct i
+    /// This struct is a wrapper for the `ServerConfig` in order to keep the `spec.server.roleConfig` setting consistent.
+    /// It is required since Spark Connect does not utilize the Stackable `Role` and therefore does not offer a `roleConfig`.
     #[derive(Clone, Debug, Default, JsonSchema, PartialEq, Serialize, Deserialize)]
     pub struct SparkConnectServerConfigWrapper {
         #[serde(flatten)]
         pub config: Option<CommonConfiguration<ServerConfigFragment, JavaCommonConfig>>,
-
+        #[serde(default)]
         pub role_config: SparkConnectServerRoleConfig,
     }
 
+    /// Global role config settings for the Spark Connect Server.
     #[derive(Clone, Debug, JsonSchema, PartialEq, Serialize, Deserialize)]
     pub struct SparkConnectServerRoleConfig {
         /// This field controls which [ListenerClass](DOCS_BASE_URL_PLACEHOLDER/listener-operator/listenerclass.html) is used to expose the Spark services.