fix: move graceful shutdown to build module

Author: maltesander

rust/operator-binary/src/controller.rs

@@ -82,7 +82,7 @@ use crate::{
         LISTENER_VOLUME_DIR, LISTENER_VOLUME_NAME, build_group_listener, build_group_listener_pvc,
         group_listener_name, secret_volume_listener_scope,
     },
-    operations::{add_graceful_shutdown_config, pdb::add_pdbs},
+    operations::pdb::add_pdbs,
     service::{build_rolegroup_headless_service, build_rolegroup_metrics_service},
 };
 
@@ -204,7 +204,7 @@ pub enum Error {
 
     #[snafu(display("failed to configure graceful shutdown"))]
     GracefulShutdown {
-        source: crate::operations::graceful_shutdown::Error,
+        source: build::graceful_shutdown::Error,
     },
 
     #[snafu(display("failed to get required Labels"))]
@@ -639,7 +639,7 @@ fn build_rolegroup_statefulset(
             &mut cb_trino,
         )
         .context(InvalidAuthenticationConfigSnafu)?;
-    add_graceful_shutdown_config(
+    build::graceful_shutdown::add_graceful_shutdown_config(
         trino,
         trino_role,
         merged_config,

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

@@ -1,7 +1,9 @@
 //! Builders that turn a `ValidatedCluster` into Kubernetes resource contents.
 //!
 //! `properties` renders the Trino `.properties` files; `config_map` assembles
-//! the per-rolegroup ConfigMap.
+//! the per-rolegroup ConfigMap; `graceful_shutdown` contributes graceful-shutdown
+//! `config.properties` entries and Pod lifecycle configuration.
 
 pub mod config_map;
+pub mod graceful_shutdown;
 pub mod properties;

…nary/src/operations/graceful_shutdown.rs …rc/controller/build/graceful_shutdown.rsrust/operator-binary/src/operations/graceful_shutdown.rs renamed to rust/operator-binary/src/controller/build/graceful_shutdown.rs rust/operator-binary/src/operations/graceful_shutdown.rs renamed to rust/operator-binary/src/controller/build/graceful_shutdown.rs

@@ -216,7 +216,9 @@ pub fn build(
     }
 
     // Graceful shutdown.
-    for (k, v) in crate::operations::graceful_shutdown_config_properties(cluster, role) {
+    for (k, v) in crate::controller::build::graceful_shutdown::graceful_shutdown_config_properties(
+        cluster, role,
+    ) {
         props.insert(k, v);
     }
 

rust/operator-binary/src/controller/build/properties/config_properties.rs

@@ -1,4 +1 @@
-pub mod graceful_shutdown;
 pub mod pdb;
-
-pub use graceful_shutdown::*;