Increase the size limit of the log volume (#383)

# Description

Increase the size limit of the log volume.

Author: siegfriedweber

CHANGELOG.md

@@ -12,7 +12,7 @@
 
 ### Changed
 
-`operator-rs` `0.40.2` -> `0.42.2` ([#360], [#371]).
+`operator-rs` `0.40.2` -> `0.44.0` ([#360], [#371], [#383]).
 - Use 0.0.0-dev product images for testing ([#361]).
 - Use testing-tools 0.2.0 ([#361]).
 - Added kuttl test suites ([#373]).
@@ -21,6 +21,7 @@
 ### Fixed
 
 - Operator now errors out when `credentialsSecret` is missing ([#375]).
+- Increase the size limit of the log volume ([#383]).
 
 [#360]: https://github.com/stackabletech/superset-operator/pull/360
 [#361]: https://github.com/stackabletech/superset-operator/pull/361
@@ -32,6 +33,7 @@
 [#375]: https://github.com/stackabletech/superset-operator/pull/375
 [#377]: https://github.com/stackabletech/superset-operator/pull/377
 [#379]: https://github.com/stackabletech/superset-operator/pull/379
+[#383]: https://github.com/stackabletech/superset-operator/pull/383
 
 ## [23.4.0] - 2023-04-17
 

Cargo.lock

@@ -11,7 +11,7 @@ publish = false
 [dependencies]
 serde = "1.0"
 serde_json = "1.0"
-stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "0.42.2" }
+stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "0.44.0" }
 strum = { version = "0.24", features = ["derive"] }
 snafu = "0.7"
 tracing = "0.1"

rust/crd/Cargo.toml

@@ -20,6 +20,7 @@ use stackable_operator::{
     config::{fragment, fragment::Fragment, fragment::ValidationError, merge::Merge},
     k8s_openapi::apimachinery::pkg::api::resource::Quantity,
     kube::{runtime::reflector::ObjectRef, CustomResource, ResourceExt},
+    memory::{BinaryMultiple, MemoryQuantity},
     product_config::flask_app_config_writer::{FlaskAppConfigOptions, PythonType},
     product_config_utils::{ConfigError, Configuration},
     product_logging::{self, spec::Logging},
@@ -36,7 +37,10 @@ pub const LOG_CONFIG_DIR: &str = "/stackable/log_config";
 pub const LOG_DIR: &str = "/stackable/log";
 pub const PYTHONPATH: &str = "/stackable/app/pythonpath";
 pub const SUPERSET_CONFIG_FILENAME: &str = "superset_config.py";
-pub const LOG_VOLUME_SIZE_IN_MIB: u32 = 10;
+pub const MAX_LOG_FILES_SIZE: MemoryQuantity = MemoryQuantity {
+    value: 10.0,
+    unit: BinaryMultiple::Mebi,
+};
 
 #[derive(Debug, Snafu)]
 pub enum Error {

rust/crd/src/lib.rs

@@ -16,13 +16,13 @@ futures = { version = "0.3", features = ["compat"] }
 indoc = "1.0"
 serde = "1.0"
 snafu = "0.7"
-stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "0.42.2" }
+stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "0.44.0" }
 stackable-superset-crd = { path = "../crd" }
 strum = { version = "0.24", features = ["derive"] }
 tokio = { version = "1.25", features = ["macros", "rt-multi-thread"] }
 tracing = "0.1"
 
 [build-dependencies]
 built = { version =  "0.6", features = ["chrono", "git2"] }
-stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "0.42.2" }
+stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "0.44.0" }
 stackable-superset-crd = { path = "../crd" }

rust/operator-binary/Cargo.toml

@@ -1,14 +1,15 @@
 use stackable_operator::{
     builder::VolumeBuilder,
-    k8s_openapi::{
-        api::core::v1::{ConfigMapVolumeSource, EmptyDirVolumeSource, Volume},
-        apimachinery::pkg::api::resource::Quantity,
-    },
-    product_logging::spec::{
-        ConfigMapLogConfig, ContainerLogConfig, ContainerLogConfigChoice, CustomContainerLogConfig,
+    k8s_openapi::api::core::v1::{ConfigMapVolumeSource, EmptyDirVolumeSource, Volume},
+    product_logging::{
+        self,
+        spec::{
+            ConfigMapLogConfig, ContainerLogConfig, ContainerLogConfigChoice,
+            CustomContainerLogConfig,
+        },
     },
 };
-use stackable_superset_crd::LOG_VOLUME_SIZE_IN_MIB;
+use stackable_superset_crd::MAX_LOG_FILES_SIZE;
 
 pub const CONFIG_VOLUME_NAME: &str = "config";
 pub const LOG_CONFIG_VOLUME_NAME: &str = "log-config";
@@ -29,7 +30,9 @@ pub fn create_volumes(
         name: LOG_VOLUME_NAME.into(),
         empty_dir: Some(EmptyDirVolumeSource {
             medium: None,
-            size_limit: Some(Quantity(format!("{LOG_VOLUME_SIZE_IN_MIB}Mi"))),
+            size_limit: Some(product_logging::framework::calculate_log_volume_size_limit(
+                &[MAX_LOG_FILES_SIZE],
+            )),
         }),
         ..Volume::default()
     });