vikunja
/
helm-chart
2
2
Fork 5
Code Issues 7 Pull Requests 1 Packages Projects Releases 1 Wiki Activity

Utilize bjw-s's common Helm library. #13

Merged
konrad merged 41 commits from :new-helm-chart into main 2023-11-07 14:47:24 +00:00
Conversation 14 Commits 41 Files Changed 24 +5 -12
1 changed files with 5 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 6cefa2e3d2 - Show all commits

17
templates/api.yaml
View File

@ -38,18 +38,6 @@ probes:
startup:
enabled: false
{{ if and .Values.frontend.ingress.enabled .Values.api.configMaps.config.enabled}}
configMaps:
# The configuration for Vikunja's api.
# https://vikunja.io/docs/config-options/
config:
enabled: true
data:
config.yml:
service:
# Vikunja needs to know the frontend URL for password reset emails.
frontendUrl: "http://{{ index .Values.frontend.ingress.main.hosts 0 "host" }}{{ index .Values.frontend.ingress.main.hosts 0 "path" }}"
{{ end }}
env:
{{ if .Values.redis.enabled }}
konrad marked this conversation as resolved Outdated
konrad commented 2023-10-11 08:35:32 +00:00
Outdated
Review
Copy Link

Nitpick, but please use "api" here instead of "backend".

Nitpick, but please use "api" here instead of "backend".
perfectra1n commented 2023-10-11 13:54:08 +00:00
Outdated
Review
Copy Link

Done:
76b209a70c :)

Done: https://kolaente.dev/vikunja/helm-chart/commit/76b209a70c3b73b8bcaeff153e8c5a56a3901975 :)
@ -58,6 +46,11 @@ env:
{{ if .Values.typesense.enabled }}
VIKUNJA_TYPESENSE_ENABLED: "true"
{{ end }}
{{ if and .Values.frontend.ingress.enabled .Values.api.configMaps.config.enabled}}
# The configuration for Vikunja's api.
# https://vikunja.io/docs/config-options/
VIKUNJA_SERVICE_FRONTENDURL: "http://{{ index .Values.frontend.ingress.main.hosts 0 "host" }}{{ index .Values.frontend.ingress.main.hosts 0 "path" }}"
{{ end }}
# Logic to decide what the api URL should be
{{ if .Values.frontend.ingress.enabled }}