From e1133ea114c5e8f1daf1d167f435d3917f47e49e Mon Sep 17 00:00:00 2001
From: Willem <wrolleman@nationtech.io>
Date: Fri, 2 May 2025 15:02:50 -0400
Subject: [PATCH] use default database_size None in LampConfig to default to
 value from helm chart

---
 harmony/src/modules/lamp.rs | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/harmony/src/modules/lamp.rs b/harmony/src/modules/lamp.rs
index 7d2b28d..47d6ca9 100644
--- a/harmony/src/modules/lamp.rs
+++ b/harmony/src/modules/lamp.rs
@@ -18,8 +18,8 @@ use crate::{
     data::{Id, Version},
     interpret::{Interpret, InterpretError, InterpretName, InterpretStatus, Outcome},
     inventory::Inventory,
-modules::k8s::deployment::K8sDeploymentScore,
-score::Score,
+    modules::k8s::deployment::K8sDeploymentScore,
+    score::Score,
     topology::{K8sclient, Topology, Url},
 };
 
@@ -37,7 +37,7 @@ pub struct LAMPScore {
 pub struct LAMPConfig {
     pub project_root: PathBuf,
     pub ssl_enabled: bool,
-    pub database_size: String,
+    pub database_size: Option<String>,
 }
 
 impl Default for LAMPConfig {
@@ -45,7 +45,7 @@ impl Default for LAMPConfig {
         LAMPConfig {
             project_root: Path::new("./src").to_path_buf(),
             ssl_enabled: true,
-            database_size: "2Gi".to_string(),
+            database_size: None,
         }
     }
 }
@@ -159,11 +159,13 @@ impl LAMPInterpret {
         inventory: &Inventory,
         topology: &T,
     ) -> Result<Outcome, InterpretError> {
-        let mut mariadb_overrides = HashMap::new();
-        mariadb_overrides.insert(
-            NonBlankString::from_str("primary.persistence.size").unwrap(),
-            self.score.config.database_size.clone(),
-        );
+        let mut values_overrides = HashMap::new();
+        if let Some(database_size) = self.score.config.database_size.clone() {
+            values_overrides.insert(
+                NonBlankString::from_str("primary.persistence.size").unwrap(),
+                database_size,
+            );
+        }
         let score = HelmChartScore {
             namespace: self.get_namespace(),
             release_name: NonBlankString::from_str(&format!("{}-database", self.score.name))
@@ -173,7 +175,7 @@ impl LAMPInterpret {
             )
             .unwrap(),
             chart_version: None,
-            values_overrides: Some(mariadb_overrides),
+            values_overrides: Some(values_overrides),
             create_namespace: true,
             install_only: true,
             values_yaml: None,
@@ -181,7 +183,6 @@ impl LAMPInterpret {
 
         score.create_interpret().execute(inventory, topology).await
     }
-
     fn build_dockerfile(&self, score: &LAMPScore) -> Result<PathBuf, Box<dyn std::error::Error>> {
         let mut dockerfile = Dockerfile::new();