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 +15
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 4136286a1a - Show all commits

15
templates/api.yaml
View File

@ -22,6 +22,21 @@ persistence:
mountPath: /etc/vikunja/config.yml
subPath: config.yml
probes:
liveness: &probes
enabled: true
custom: true
spec:
httpGet:
path: /api/v1
konrad marked this conversation as resolved Outdated
konrad commented 2023-10-10 16:38:16 +00:00
Outdated
Review
Copy Link

Shouldn't this use /api/v1/info as /api/v1 will usually give an error?

Shouldn't this use `/api/v1/info` as `/api/v1` will usually give an error?
perfectra1n commented 2023-10-10 22:32:46 +00:00
Outdated
Review
Copy Link

LOL, I've gotta say, that explains a lot about my own Vikunja deployment after realizing that /api/v1 returns a 401...

fixed!

LOL, I've gotta say, that explains a lot about my own Vikunja deployment after realizing that `/api/v1` returns a `401`... fixed!
perfectra1n commented 2023-10-10 22:38:34 +00:00
Outdated
Review
Copy Link

61b8892ff0

https://kolaente.dev/vikunja/helm-chart/commit/61b8892ff0aa802e33c40656e27caa6105b48881
konrad commented 2023-10-11 08:33:32 +00:00
Outdated
Review
Copy Link

hehe yeah there's an item in the backlog to add a health endpoint eventually which would also check if the db connection works etc.

hehe yeah there's an item in the backlog to add a `health` endpoint eventually which would also check if the db connection works etc.
port: http
initialDelaySeconds: 0
periodSeconds: 10
timeoutSeconds: 1
failureThreshold: 3
readiness: *probes
startup:
enabled: false
env:
{{ if .Values.redis.enabled }}