fix: use upstream STACKABLE_LOG_DIR

Author: maltesander

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

@@ -9,14 +9,16 @@ use stackable_operator::{
     k8s_openapi::api::core::v1::EnvVar,
     kube::ResourceExt,
     product_logging::framework::create_vector_shutdown_file_command,
-    v2::builder::pod::container::EnvVarSet,
+    v2::{
+        builder::pod::container::EnvVarSet,
+        product_logging::framework::STACKABLE_LOG_DIR,
+    },
 };
 
 use crate::{
     controller::{ValidatedCluster, ValidatedLogging},
     crd::{
-        AirflowExecutor, AirflowRole, HTTP_PORT, LOG_CONFIG_DIR, STACKABLE_LOG_DIR,
-        TEMPLATE_LOCATION, TEMPLATE_NAME,
+        AirflowExecutor, AirflowRole, HTTP_PORT, LOG_CONFIG_DIR, TEMPLATE_LOCATION, TEMPLATE_NAME,
         authentication::{
             AirflowAuthenticationClassResolved, AirflowClientAuthenticationDetailsResolved,
         },

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

@@ -8,7 +8,10 @@ use stackable_operator::{
     builder::configmap::ConfigMapBuilder,
     k8s_openapi::api::core::v1::ConfigMap,
     product_logging::framework::VECTOR_CONFIG_FILE,
-    v2::types::operator::{RoleGroupName, RoleName},
+    v2::{
+        product_logging::framework::STACKABLE_LOG_DIR,
+        types::operator::{RoleGroupName, RoleName},
+    },
 };
 
 use crate::{
@@ -19,7 +22,7 @@ use crate::{
             webserver_config,
         },
     },
-    crd::{AIRFLOW_CONFIG_FILENAME, AirflowConfigOverrides, Container, STACKABLE_LOG_DIR},
+    crd::{AIRFLOW_CONFIG_FILENAME, AirflowConfigOverrides, Container},
 };
 
 #[derive(Snafu, Debug)]

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

@@ -16,7 +16,10 @@ use stackable_operator::{
         api::core::v1::{ConfigMap, PodTemplateSpec},
     },
     kvp::{Label, LabelError},
-    v2::builder::pod::container::new_container_builder,
+    v2::{
+        builder::pod::container::new_container_builder,
+        product_logging::framework::STACKABLE_LOG_DIR,
+    },
 };
 
 use crate::{
@@ -35,7 +38,7 @@ use crate::{
             volumes::{self, CONFIG_VOLUME_NAME, LOG_CONFIG_VOLUME_NAME, LOG_VOLUME_NAME},
         },
     },
-    crd::{CONFIG_PATH, Container, LOG_CONFIG_DIR, STACKABLE_LOG_DIR, TEMPLATE_NAME},
+    crd::{CONFIG_PATH, Container, LOG_CONFIG_DIR, TEMPLATE_NAME},
 };
 
 #[derive(Snafu, Debug)]

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

@@ -24,6 +24,7 @@ use stackable_operator::{
             container::new_container_builder,
             volume::{ListenerReference, listener_operator_volume_source_builder_build_pvc},
         },
+        product_logging::framework::STACKABLE_LOG_DIR,
         types::operator::RoleGroupName,
     },
 };
@@ -47,7 +48,7 @@ use crate::{
     crd::{
         AirflowExecutor, AirflowRole, CONFIG_PATH, Container, HTTP_PORT_NAME, LISTENER_PVC_NAME,
         LISTENER_VOLUME_DIR, LOG_CONFIG_DIR, METRICS_CONTAINER_NAME, METRICS_PORT,
-        METRICS_PORT_NAME, STACKABLE_LOG_DIR, TEMPLATE_LOCATION, TEMPLATE_VOLUME_NAME,
+        METRICS_PORT_NAME, TEMPLATE_LOCATION, TEMPLATE_VOLUME_NAME,
     },
 };
 

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

@@ -39,6 +39,7 @@ use stackable_operator::{
     v2::{
         config_overrides::KeyValueConfigOverrides,
         flask_config_writer::{FlaskAppConfigOptions, PythonType},
+        product_logging::framework::STACKABLE_LOG_DIR,
         role_utils::GenericCommonConfig,
         types::{
             common::Port,
@@ -77,7 +78,6 @@ pub const APP_NAME: &str = "airflow";
 pub const FIELD_MANAGER: &str = "airflow-operator";
 pub const OPERATOR_NAME: &str = "airflow.stackable.tech";
 pub const CONFIG_PATH: &str = "/stackable/app/config";
-pub const STACKABLE_LOG_DIR: &str = "/stackable/log";
 pub const LOG_CONFIG_DIR: &str = "/stackable/app/log_config";
 pub const AIRFLOW_HOME: &str = "/stackable/airflow";
 pub const AIRFLOW_CONFIG_FILENAME: &str = "webserver_config.py";