NationTech/harmony
6
0
Fork 1
Code Issues 32 Pull Requests 7 Actions 7 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 +23 -2
wjro commented 2025-05-02 16:01:12 +00:00
Owner
Copy Link
No description provided.
wjro added 1 commit 2025-05-02 16:01:13 +00:00
johnride requested changes 2025-05-02 16:05:17 +00:00
johnride left a comment
Owner
Copy Link

Good, je changerais le default pour un meilleur DX mais sinon LGTM

Good, je changerais le default pour un meilleur DX mais sinon LGTM
harmony/src/modules/lamp.rs Outdated
@ -43,6 +45,7 @@ impl Default for LAMPConfig {
LAMPConfig {
project_root: Path::new("./src").to_path_buf(),
ssl_enabled: true,
database_size: "2Gi".to_string(),
johnride commented 2025-05-02 16:04:56 +00:00
Owner
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.
wjro added 2 commits 2025-05-02 19:07:44 +00:00
johnride approved these changes 2025-05-02 19:09:32 +00:00
johnride approved these changes 2025-05-02 19:59:32 +00:00
wjro merged commit 764fd6d451 into master 2025-05-03 00:20:47 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
johnride
Labels
Clear labels   
Architecture Decision Record

   
Good first issue

   
RFC
No Label
Architecture Decision Record
Good first issue
RFC
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: NationTech/harmony#28
No description provided.
Delete Branch "lamp-env-vars"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?