NationTech/harmony
6
0
Fork 1
Code Issues 35 Pull Requests 6 Actions Packages Projects Releases 1 Wiki Activity

chore: added default mariadb size and pass env variables to php app #28

Merged
wjro merged 3 commits from lamp-env-vars into master 2025-05-03 00:20:47 +00:00
Conversation 0 Commits 3 Files Changed 2 +12 -11
1 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e1133ea114 - Show all commits

23
harmony/src/modules/lamp.rs
View File

@ -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,
johnride commented 2025-05-02 16:04:56 +00:00
Outdated
Review
Copy Link

Maintenant que je vois ca je pense que ca devrait etre un Option qui est None par default pour deleguer la tache de set le default a la helm chart. Comme ca nos utilisateurs n'auront pas de surprise et c'est un detail de moins a gerer par nous-meme.

Par exemple, si dans 10 ans le default est rendu a 100G parce que la structure de la BD a change ou je sais pas quoi, on n'aura pas besoin de s'en occuper nous-memes.

Maintenant que je vois ca je pense que ca devrait etre un Option<String> qui est None par default pour deleguer la tache de set le default a la helm chart. Comme ca nos utilisateurs n'auront pas de surprise et c'est un detail de moins a gerer par nous-meme. Par exemple, si dans 10 ans le default est rendu a 100G parce que la structure de la BD a change ou je sais pas quoi, on n'aura pas besoin de s'en occuper nous-memes.
}
}
}
@ -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();