Skip to content

Commit cb850dc

Browse files
adwk67claude
andcommitted
refactor: move rolegroup ConfigMap build into controller/build/config_map
Relocates build_rolegroup_config_map (plus the jaas_config_file helper and its Error enum) from resource/configmap.rs to controller/build/config_map.rs, colocating the ConfigMap assembler with the per-file property builders and matching the controller/build/config_map.rs layout in hdfs/airflow. Pure move + repointed caller; no behaviour change (18 tests pass). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent c067058 commit cb850dc

4 files changed

Lines changed: 3 additions & 4 deletions

File tree

rust/operator-binary/src/controller.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ use crate::{
4040
discovery::{self, build_discovery_configmap},
4141
operations::pdb::add_pdbs,
4242
resource::{
43-
configmap::build_rolegroup_config_map,
4443
listener::build_broker_rolegroup_bootstrap_listener,
4544
service::{build_rolegroup_headless_service, build_rolegroup_metrics_service},
4645
statefulset::{build_broker_rolegroup_statefulset, build_controller_rolegroup_statefulset},
@@ -167,7 +166,7 @@ pub enum Error {
167166

168167
#[snafu(display("failed to build configmap"))]
169168
BuildConfigMap {
170-
source: crate::resource::configmap::Error,
169+
source: crate::controller::build::config_map::Error,
171170
},
172171

173172
#[snafu(display("failed to build service"))]
@@ -329,7 +328,7 @@ pub async fn reconcile_kafka(
329328
)
330329
.context(BuildPodDescriptorsSnafu)?;
331330

332-
let rg_configmap = build_rolegroup_config_map(
331+
let rg_configmap = build::config_map::build_rolegroup_config_map(
333332
kafka,
334333
&image,
335334
&kafka_security,
File renamed without changes.
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
//! Builders that assemble Kubernetes resources for kafka rolegroups.
22
3+
pub mod config_map;
34
pub mod properties;
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
pub mod configmap;
21
pub mod listener;
32
pub mod service;
43
pub mod statefulset;

0 commit comments

Comments
 (0)