johnride
  • Joined on 2024-02-06
johnride commented on pull request NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

Using a macro here is weird at the very least.

johnride suggested changes for NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

Good improvement over the first version, still some work to do to have it really look good. Keep it up!

johnride commented on pull request NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

No need for this monitoring_stack: Stack field for now. This is a case of YAGNI. We support only one type and I don't see in the very short term a use case that would force us.

johnride commented on pull request NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

What is service_name used for? As this is user facing, a bit of rust doc (with triple slashes ///) to describe how to use would be useful here.

johnride commented on pull request NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

Poor function name. What's the use of this function?

johnride commented on pull request NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

Nice, but did you test smtp? If it's not working yet there should be a // TODO comment or something like that.

johnride commented on pull request NationTech/harmony#37 2025-05-15 20:33:30 +00:00
monitoringalerting

Why use values_overrides here when there already is a values_yaml right above? Would make it a lot more readable I think.

johnride commented on pull request NationTech/harmony#32 2025-05-15 14:56:47 +00:00
feat: add ingress score

What about path, path_type and namespace? These certainly don't support any String?

johnride suggested changes for NationTech/harmony#32 2025-05-15 14:56:47 +00:00
feat: add ingress score
johnride deleted branch feat/lampOKD from NationTech/harmony 2025-05-14 15:48:59 +00:00
johnride pushed to master at NationTech/harmony 2025-05-14 15:48:58 +00:00
861f266c4e Merge pull request 'feat: LAMP stack and Monitoring stack now work on OKD, we just have to manually set a few serviceaccounts to privileged scc until we find a better solution' (#36) from feat/lampOKD into master
51724d0e55 feat: LAMP stack and Monitoring stack now work on OKD, we just have to manually set a few serviceaccounts to privileged scc until we find a better solution
Compare 2 commits »
johnride merged pull request NationTech/harmony#36 2025-05-14 15:48:57 +00:00
feat: LAMP stack and Monitoring stack now work on OKD, we just have to manually set a few serviceaccounts to privileged scc until we find a better solution
johnride pushed to feat/lampOKD at NationTech/harmony 2025-05-14 15:48:33 +00:00
51724d0e55 feat: LAMP stack and Monitoring stack now work on OKD, we just have to manually set a few serviceaccounts to privileged scc until we find a better solution
johnride pushed to feat/lampOKD at NationTech/harmony 2025-05-14 15:47:08 +00:00
ad72409287 feat: LAMP stack and Monitoring stack now work on OKD, we just have to manually set a few serviceaccounts to privileged scc until we find a better solution
c2d1cb9b76 Merge pull request 'upgrade stack size from default 1MB on windows (k3d stack overflow otherwise)' (#34) from windows-stack-size-increase into master
c84a02c8ec upgrade stack size from default 1MB on windows (k3d stack overflow otherwise)
Compare 3 commits »
johnride created pull request NationTech/harmony#36 2025-05-14 15:36:50 +00:00
feat: LAMP stack and Monitoring stack now work on OKD, we just have to manually set a few serviceaccounts to privileged scc until we find a better solution
johnride deleted branch php-fill-volume from NationTech/harmony 2025-05-14 15:35:23 +00:00
johnride deleted branch upload_folder_improvements from NationTech/harmony 2025-05-14 15:35:04 +00:00
johnride deleted branch chore/warnings from NationTech/harmony 2025-05-14 15:34:54 +00:00
johnride deleted branch ensure-helm-available from NationTech/harmony 2025-05-14 15:34:49 +00:00