Apply suggestions from code review

Co-authored-by: Razvan-Daniel Mihai <[email protected]>

Author: maltesander

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

@@ -115,7 +115,7 @@ pub mod versioned {
     /// Global role config settings for the Spark Connect Server.
     #[derive(Clone, Debug, JsonSchema, PartialEq, Serialize, Deserialize)]
     #[serde(rename_all = "camelCase")]
-    pub struct SparkConnectServerRoleConfig {
+    struct SparkConnectServerRoleConfig {
         /// This field controls which [ListenerClass](DOCS_BASE_URL_PLACEHOLDER/listener-operator/listenerclass.html)
         /// is used to expose the Spark Connect services.
         #[serde(default = "default_listener_class")]

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

@@ -633,7 +633,12 @@ pub(crate) fn build_listener(
     role_config: &v1alpha1::SparkConnectServerRoleConfig,
     resolved_product_image: &ResolvedProductImage,
 ) -> Result<listener::v1alpha1::Listener, Error> {
-    let listener_name = format!("{cluster}-server", cluster = scs.name_any());
+    let listener_name = format!(
+        "{cluster}-{role}",
+        cluster = scs.name_any(),
+        role = SparkConnectRole::Server
+    );
+
 
     let listener_class = role_config.listener_class.clone();
     let role = SparkConnectRole::Server.to_string();