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

2
templates/api.yaml
View File

@ -28,7 +28,7 @@ probes:
custom: true
spec:
httpGet:
path: /api/v1
path: /api/v1/info
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