remove result as return value as it is now used in calling function

Author: adwk67

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

@@ -98,10 +98,7 @@ pub fn build_rolegroup_config_map(
         &Container::Vector,
         &mut cm_builder,
         &validated_cluster.image,
-    )
-    .context(InvalidLoggingConfigSnafu {
-        cm_name: rolegroup.object_name(),
-    })?;
+    );
 
     cm_builder.build().with_context(|_| BuildConfigMapSnafu {
         rolegroup: rolegroup.clone(),

rust/operator-binary/src/product_logging.rs

@@ -32,8 +32,6 @@ pub enum Error {
     MissingVectorAggregatorAddress,
 }
 
-type Result<T, E = Error> = std::result::Result<T, E>;
-
 const LOG_CONFIG_FILE: &str = "log_config.py";
 const LOG_FILE: &str = "airflow.py.json";
 
@@ -45,8 +43,7 @@ pub fn extend_config_map_with_log_config<C, K>(
     vector_container: &C,
     cm_builder: &mut ConfigMapBuilder,
     resolved_product_image: &ResolvedProductImage,
-) -> Result<()>
-where
+) where
     C: Clone + Ord + Display,
     K: Resource,
 {
@@ -76,8 +73,6 @@ where
             product_logging::framework::create_vector_config(rolegroup, vector_log_config),
         );
     }
-
-    Ok(())
 }
 
 fn create_airflow_config(