chore: Apply rustfmt and add CHANGELOG entry for product-config removal

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

Author: maltesander

CHANGELOG.md

@@ -18,6 +18,7 @@ All notable changes to this project will be documented in this file.
   This struct is consistent between different CRDs, so that you can easily copy/paste it between stacklets.
   Read on the [Hive database documentation](https://docs.stackable.tech/home/nightly/hive/usage-guide/database-driver) for details ([#674]).
 - Internal operator refactoring: introduce dereference() and validate() steps in the reconciler ([#707]).
+- Remove the direct `product-config` dependency; `hive-site.xml`, `security.properties` and `core-site.xml` are now rendered by typed Rust builders. Switch `stackable-operator` to the `smooth-operator` branch and adopt its shared `v2::` framework ([#XXX]).
 - test: Bump vector-aggregator to 0.55.0, replace /graphql call with gRPC call ([#713]).
 - Deprecated support for `4.0.1` ([#711]).
 

rust/operator-binary/src/controller.rs

@@ -436,12 +436,11 @@ pub async fn reconcile_hive(
                     rolegroup: rolegroup.clone(),
                 })?;
 
-            cluster_resources
-                .add(client, rg_configmap)
-                .await
-                .context(ApplyRoleGroupConfigSnafu {
+            cluster_resources.add(client, rg_configmap).await.context(
+                ApplyRoleGroupConfigSnafu {
                     rolegroup: rolegroup.clone(),
-                })?;
+                },
+            )?;
 
             // Note: The StatefulSet needs to be applied after all ConfigMaps and Secrets it
             // mounts to prevent unnecessary Pod restarts.

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

@@ -10,8 +10,9 @@ use stackable_operator::{
 
 use crate::{
     controller::{
-        ValidatedCluster, build_recommended_labels,
+        ValidatedCluster,
         build::properties::{core_site, hive_site, resolved_overrides, security_properties},
+        build_recommended_labels,
     },
     crd::{CORE_SITE_XML, HIVE_SITE_XML, HiveRole, JVM_SECURITY_PROPERTIES_FILE, v1alpha1},
     framework::writer::{to_hadoop_xml, to_java_properties_string},

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

@@ -28,7 +28,8 @@ mod tests {
     use super::*;
 
     fn hive_cluster(yaml: &str) -> v1alpha1::HiveCluster {
-        stackable_operator::utils::yaml_from_str_singleton_map(yaml).expect("valid HiveCluster YAML")
+        stackable_operator::utils::yaml_from_str_singleton_map(yaml)
+            .expect("valid HiveCluster YAML")
     }
 
     const NO_KERBEROS_YAML: &str = r#"

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

@@ -11,10 +11,8 @@ use std::collections::BTreeMap;
 
 use snafu::{ResultExt, Snafu};
 use stackable_operator::{
-    crd::s3,
-    database_connections::drivers::jdbc::JdbcDatabaseConnectionDetails,
-    k8s_openapi::api::core::v1::EnvVar,
-    utils::cluster_info::KubernetesClusterInfo,
+    crd::s3, database_connections::drivers::jdbc::JdbcDatabaseConnectionDetails,
+    k8s_openapi::api::core::v1::EnvVar, utils::cluster_info::KubernetesClusterInfo,
 };
 
 use crate::{
@@ -103,7 +101,11 @@ pub fn build(
     if let Some(s3) = s3_connection_spec {
         data.insert(
             MetaStoreConfig::S3_ENDPOINT.to_string(),
-            Some(s3.endpoint().context(ConfigureS3ConnectionSnafu)?.to_string()),
+            Some(
+                s3.endpoint()
+                    .context(ConfigureS3ConnectionSnafu)?
+                    .to_string(),
+            ),
         );
         data.insert(
             MetaStoreConfig::S3_REGION_NAME.to_string(),
@@ -163,7 +165,8 @@ mod tests {
     use super::*;
 
     fn hive_cluster(yaml: &str) -> v1alpha1::HiveCluster {
-        stackable_operator::utils::yaml_from_str_singleton_map(yaml).expect("valid HiveCluster YAML")
+        stackable_operator::utils::yaml_from_str_singleton_map(yaml)
+            .expect("valid HiveCluster YAML")
     }
 
     const DERBY_YAML: &str = r#"

rust/operator-binary/src/crd/mod.rs

@@ -19,7 +19,6 @@ use stackable_operator::{
         fragment::{self, Fragment, ValidationError},
         merge::Merge,
     },
-    v2::config_overrides::KeyValueConfigOverrides,
     crd::s3,
     deep_merger::ObjectOverrides,
     k8s_openapi::apimachinery::pkg::api::resource::Quantity,
@@ -30,6 +29,7 @@ use stackable_operator::{
     shared::time::Duration,
     status::condition::{ClusterCondition, HasStatusCondition},
     utils::cluster_info::KubernetesClusterInfo,
+    v2::config_overrides::KeyValueConfigOverrides,
     versioned::versioned,
 };
 use strum::{Display, EnumIter, EnumString, IntoEnumIterator};