feat: Enable restart controller (#999)

* feat: Enable restart controller

* changelog

* Use constant

* Use op-rs 0.102.0

Author: sbernauer

CHANGELOG.md

@@ -9,9 +9,11 @@ All notable changes to this project will be documented in this file.
 - Add `ZOOCFGDIR` env var to `/stackable/rwconfig` to improve shell script usage like `zkCleanup.sh` ([#988]).
 - Support objectOverrides using `.spec.objectOverrides`.
   See [objectOverrides concepts page](https://docs.stackable.tech/home/nightly/concepts/overrides/#object-overrides) for details ([#987]).
+- Enable the [restart-controller](https://docs.stackable.tech/home/nightly/commons-operator/restarter/), so that the Pods are automatically restarted on config changes ([#999]).
 
 [#987]: https://github.com/stackabletech/zookeeper-operator/pull/987
 [#988]: https://github.com/stackabletech/zookeeper-operator/pull/988
+[#999]: https://github.com/stackabletech/zookeeper-operator/pull/999
 
 ## [25.11.0] - 2025-11-07
 

Cargo.lock

@@ -11,7 +11,7 @@ repository = "https://github.com/stackabletech/zookeeper-operator"
 
 [workspace.dependencies]
 product-config = { git = "https://github.com/stackabletech/product-config.git", tag = "0.8.0" }
-stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.101.2", features = ["telemetry", "versioned"] }
+stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.102.0", features = ["telemetry", "versioned"] }
 
 anyhow = "1.0"
 built = { version = "0.8", features = ["chrono", "git2"] }

Cargo.nix

@@ -33,6 +33,7 @@ use stackable_operator::{
         product_image_selection::{self, ResolvedProductImage},
         rbac::build_rbac_resources,
     },
+    constants::RESTART_CONTROLLER_ENABLED_LABEL,
     k8s_openapi::{
         DeepMerge,
         api::{
@@ -486,6 +487,9 @@ pub async fn reconcile_zk(
                 rolegroup: rolegroup.clone(),
             })?;
 
+        // Note: The StatefulSet needs to be applied after all ConfigMaps and Secrets it mounts
+        // to prevent unnecessary Pod restarts.
+        // See https://github.com/stackabletech/commons-operator/issues/111 for details.
         ss_cond_builder.add(
             cluster_resources
                 .add(client, rg_statefulset)
@@ -1019,10 +1023,8 @@ fn build_server_rolegroup_statefulset(
             &rolegroup_ref.role,
             &rolegroup_ref.role_group,
         ))
-        // The initial restart muddles up the integration tests. This can be re-enabled as soon
-        // as https://github.com/stackabletech/commons-operator/issues/111 is implemented.
-        // .with_label("restarter.stackable.tech/enabled", "true")
         .context(ObjectMetaSnafu)?
+        .with_label(RESTART_CONTROLLER_ENABLED_LABEL.to_owned())
         .build();
 
     let statefulset_match_labels =

Cargo.toml

@@ -9,6 +9,7 @@ apiVersion: apps/v1
 kind: StatefulSet
 metadata:
   name: test-zk-server-primary
+  generation: 1 # There should be no unneeded Pod restarts
 spec:
   template:
     spec: