add comments, remove todo

Author: maltesander

rust/operator-binary/src/config/command.rs

@@ -175,20 +175,16 @@ pub fn controller_kafka_container_command(
 }
 
 fn to_listeners(port: u16) -> String {
-    // TODO:
-    // - document that variables are set in stateful set
-    // - customize listener (CONTROLLER / CONTROLLER_AUTH?)
+    // The environment variables are set in the statefulset of the controller
     format!(
         "{listener_name}://$POD_NAME.$ROLEGROUP_REF.$NAMESPACE.svc.$CLUSTER_DOMAIN:{port}",
         listener_name = KafkaListenerName::Controller
     )
 }
 
 fn to_listener_security_protocol_map(kafka_listeners: &KafkaListenerConfig) -> String {
-    // TODO: make configurable - CONTROLLER_AUTH
     kafka_listeners
         .listener_security_protocol_map_for_listener(&KafkaListenerName::Controller)
-        // todo better error
         .unwrap_or("".to_string())
 }
 

rust/operator-binary/src/crd/role/controller.rs

@@ -42,7 +42,6 @@ pub const CONTROLLER_PROPERTIES_FILE: &str = "controller.properties";
 #[strum(serialize_all = "kebab-case")]
 pub enum ControllerContainer {
     Vector,
-    // TODO: Kafka, Kraft, Controller?
     Kafka,
 }