chore: Update changelog

Author: siegfriedweber

crates/stackable-operator/CHANGELOG.md

@@ -8,13 +8,16 @@ All notable changes to this project will be documented in this file.
 
 - Add `Client::{get_feature_gates,get_enabled_feature_gates,get_disabled_feature_gates}` associated
   functions to retrieve all, enabled, or disabled feature gates from the Kubernetes apiserver ([#1207]).
+- Support the annotation `secrets.stackable.tech/backend.autotls.cert.domain-components-in-subject-dn`
+  in the `SecretOperatorVolumeSourceBuilder` ([#1209]).
 
 ### Changed
 
 - BREAKING: Use `serde_json::Value` instead of `String` for user-provided JSON `configOverrides`. This change is marked as breaking, as it causes a breaking change to the CRDs ([#1206]).
 
 [#1206]: https://github.com/stackabletech/operator-rs/pull/1206
 [#1207]: https://github.com/stackabletech/operator-rs/pull/1207
+[#1209]: https://github.com/stackabletech/operator-rs/pull/1209
 
 ## [0.111.1] - 2026-04-28
 

crates/stackable-operator/src/builder/pod/volume.rs

@@ -402,10 +402,9 @@ impl SecretOperatorVolumeSourceBuilder {
         }
 
         if let Some(enabled) = self.auto_tls_cert_domain_components_in_subject_dn {
-            annotations.insert(
-                Annotation::auto_tls_cert_domain_components_in_subject_dn(enabled)
-                    .context(ParseAnnotationSnafu)?,
-            );
+            annotations.insert(Annotation::auto_tls_cert_domain_components_in_subject_dn(
+                enabled,
+            ));
         }
 
         Ok(EphemeralVolumeSource {

crates/stackable-operator/src/kvp/annotation/mod.rs

@@ -161,21 +161,23 @@ impl Annotation {
     pub fn autoscaling_retry(retry: bool) -> Self {
         // SAFETY: We use expect here, because the input parameter can only be one of two possible
         // values: true or false. This fact in combination with the known annotation key length
-        // allows use to use expect here, instead of bubbling up the error.
+        // allows us to use expect here, instead of bubbling up the error.
         let kvp = KeyValuePair::try_from(("autoscaling.stackable.tech/retry", retry.to_string()))
             .expect("autoscaling retry annotation must be valid");
         Self(kvp)
     }
 
-    /// Constructs a `secrets.stackable.tech/backend.autotls.cert.lifetime` annotation.
-    pub fn auto_tls_cert_domain_components_in_subject_dn(
-        enabled: bool,
-    ) -> Result<Self, AnnotationError> {
+    /// Constructs a `secrets.stackable.tech/backend.autotls.cert.domain-components-in-subject-dn` annotation.
+    pub fn auto_tls_cert_domain_components_in_subject_dn(enabled: bool) -> Self {
+        // SAFETY: We use expect here, because the input parameter can only be one of two possible
+        // values: true or false. This fact in combination with the known annotation key length
+        // allows us to use expect here, instead of bubbling up the error.
         let kvp = KeyValuePair::try_from((
             "secrets.stackable.tech/backend.autotls.cert.domain-components-in-subject-dn",
             enabled.to_string(),
-        ))?;
-        Ok(Self(kvp))
+        ))
+        .expect("annotation must be valid");
+        Self(kvp)
     }
 }
 
@@ -368,4 +370,13 @@ mod test {
 
         assert_eq!(annotations.len(), 2);
     }
+
+    #[test]
+    fn test_boolean_annotations() {
+        // Check that the functions do not fail for all possible inputs
+        for value in [false, true] {
+            Annotation::autoscaling_retry(value);
+            Annotation::auto_tls_cert_domain_components_in_subject_dn(value);
+        }
+    }
 }