Postgres sub-chart routing defaults to localhost instead of necessary service name #23

New Issue

MaximUltimatum · 2024-01-10T23:25:24Z

MaximUltimatum commented

2024-01-10 23:25:24 +00:00

Hello!
Thank you for your effort maintaining this chart.
I attempted to install the chart, and the api was unable to talk to the database (kept trying to reach out over localhost instead of using kubernetes routing) until I added VIKUNJA_DATABASE_HOST: project-pym-postgresql (where project-pym-postgresql is the name of my postgres service) to the env config.

Is this something that is a quirk of my distribution? (Vanilla k8s installed with kubespray). Does attempting to use localhost for the database work for other users?

I think at least it would be worth making a documentation update to set that flag in case someone runs into a similar issue in the future. I'm happy to make it! Just wanted to reach out and make sure there wasn't a deeper issue.

Hello! Thank you for your effort maintaining this chart. I attempted to install the chart, and the api was unable to talk to the database (kept trying to reach out over localhost instead of using kubernetes routing) until I added `VIKUNJA_DATABASE_HOST: project-pym-postgresql` (where project-pym-postgresql is the name of my postgres service) to the env config. Is this something that is a quirk of my distribution? (Vanilla k8s installed with kubespray). Does attempting to use localhost for the database work for other users? I think at least it would be worth making a documentation update to set that flag in case someone runs into a similar issue in the future. I'm happy to make it! Just wanted to reach out and make sure there wasn't a deeper issue.

konrad commented

2024-01-11 07:51:36 +00:00

localhost is the default of vikunja and will never work in a docker context. Iirc we don't set the value automatically in the chart because the database might be external.

I'd happily accept a PR with a clarification in the docs!

localhost is the default of vikunja and will never work in a docker context. Iirc we don't set the value automatically in the chart because the database might be external. I'd happily accept a PR with a clarification in the docs!

Sign in to join this conversation.