refactor: Move discovery ConfigMap builder into zk_controller/build

Mirror trino-operator and hdfs-operator by relocating discovery.rs to
zk_controller/build/discovery.rs. The zk_controller `build` module is now
pub(crate) so the znode controller can reach the shared builder; both
controllers build discovery ConfigMaps for a ZookeeperCluster.

Pure move: no behavior change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: maltesander

rust/operator-binary/src/main.rs

@@ -42,7 +42,6 @@ use crate::{
 mod command;
 mod config;
 pub mod crd;
-mod discovery;
 mod framework;
 mod listener;
 mod operations;

rust/operator-binary/src/zk_controller.rs

@@ -73,7 +73,6 @@ use crate::{
         security::{self, ZookeeperSecurity},
         v1alpha1::{self, ZookeeperServerRoleConfig},
     },
-    discovery::{self, build_discovery_configmap},
     listener::{build_role_listener, role_listener_name},
     operations::{graceful_shutdown::add_graceful_shutdown_config, pdb::add_pdbs},
     service::{
@@ -82,7 +81,7 @@ use crate::{
     utils::build_recommended_labels,
 };
 
-mod build;
+pub(crate) mod build;
 mod dereference;
 mod validate;
 
@@ -152,7 +151,9 @@ pub enum Error {
     },
 
     #[snafu(display("failed to build discovery ConfigMap"))]
-    BuildDiscoveryConfig { source: discovery::Error },
+    BuildDiscoveryConfig {
+        source: build::discovery::Error,
+    },
 
     #[snafu(display("failed to apply discovery ConfigMap"))]
     ApplyDiscoveryConfig {
@@ -447,7 +448,7 @@ pub async fn reconcile_zk(
     // std's SipHasher is deprecated, and DefaultHasher is unstable across Rust releases.
     // We don't /need/ stability, but it's still nice to avoid spurious changes where possible.
     let mut discovery_hash = FnvHasher::with_key(0);
-    let discovery_cm = build_discovery_configmap(
+    let discovery_cm = build::discovery::build_discovery_configmap(
         zk,
         zk,
         ZK_CONTROLLER_NAME,

rust/operator-binary/src/zk_controller/build.rs

@@ -6,4 +6,5 @@
 //! configuration.
 
 pub mod config_map;
+pub mod discovery;
 pub mod properties;

rust/operator-binary/src/discovery.rs …ary/src/zk_controller/build/discovery.rsrust/operator-binary/src/discovery.rs renamed to rust/operator-binary/src/zk_controller/build/discovery.rs rust/operator-binary/src/discovery.rs renamed to rust/operator-binary/src/zk_controller/build/discovery.rs

@@ -27,8 +27,8 @@ use tracing::{debug, info};
 use crate::{
     APP_NAME, OPERATOR_NAME,
     crd::{ZookeeperRole, security::ZookeeperSecurity, v1alpha1},
-    discovery::{self, build_discovery_configmap},
     listener::role_listener_name,
+    zk_controller::build::discovery::{self, build_discovery_configmap},
 };
 
 mod dereference;