nicholaswilde / helm-charts

My collection of Helm charts.
https://artifacthub.io/packages/search?repo=nicholaswilde
Apache License 2.0
70 stars 32 forks source link

Fix mariadb not booting #24

Closed Ornias1993 closed 3 years ago

Ornias1993 commented 3 years ago

This is an example fixed, based on how the k8s-at-home common chart handles mounting configmaps.

I've not tested it with your common chart, but with this example, you should be able to fix #19

nicholaswilde commented 3 years ago

Would you mind sharing your values.yaml file as well? I'm getting the same errors using both my common chart and k8s-at-home's

Ornias1993 commented 3 years ago

@nicholaswilde everything from @truecharts is public, but not fully native Helm conpatible out-of-the-box. The clue of this example is adding a configmap, it shouldnt require much values.yaml changes if any.

plsnotracking commented 3 years ago

@nicholaswilde I've tried this one - https://github.com/groundhog2k/helm-charts/blob/master/charts/mariadb/values.yaml. It seems to work fine (minus the loadBalancerIP sharing stuff). Not sure if it helps, I know how YAMLs work but don't really understand how to build a helm chart out of it. Maybe this helps. Thanks.

nicholaswilde commented 3 years ago

@nicholaswilde I've tried this one - https://github.com/groundhog2k/helm-charts/blob/master/charts/mariadb/values.yaml. It seems to work fine (minus the loadBalancerIP sharing stuff). Not sure if it helps, I know how YAMLs work but don't really understand how to build a helm chart out of it. Maybe this helps. Thanks.

Thanks for the info! I greatly appreciate it!

plsnotracking commented 3 years ago

Also, currently I just use https://github.com/groundhog2k/helm-charts/blob/master/charts/mariadb/values.yaml, and patch the service by executing following commands

k annotate service mariadb -n mariadb metallb.universe.tf/address-pool=default
k annotate service mariadb -n mariadb metallb.universe.tf/allow-shared-ip=common-svc

If someone was wondering how it currently works. @nicholaswilde

Ornias1993 commented 3 years ago

Also, currently I just use https://github.com/groundhog2k/helm-charts/blob/master/charts/mariadb/values.yaml, and patch the service by executing following commands

k annotate service mariadb -n mariadb metallb.universe.tf/address-pool=default
k annotate service mariadb -n mariadb metallb.universe.tf/allow-shared-ip=common-svc

If someone was wondering how it currently works. @nicholaswilde

Why are you doing all sorts of hardship using patches?! metalLB isn't even part of any of these charts, thats your specific system.

nicholaswilde commented 3 years ago

@plsnotracking did you use my chart with groundhog's values or did you use groundhog's chart?

plsnotracking commented 3 years ago

I was just mentioning it anyone who would've had a similar setup to me. It works almost identical to nicholas's chart until we have a fix for it. @Ornias1993

I use their values and their chart. @nicholaswilde

nicholaswilde commented 3 years ago

It seems that it's now working with not additional changes. It must have been something upstream. Thanks for the help with this.