wekan / charts

Wekan Helm Charts to deploy to Kubernetes
MIT License
2 stars 15 forks source link

WRITABLE_PATH /data in Helm chart #14

Closed Nightreaver closed 1 year ago

Nightreaver commented 1 year ago

Hello,

I just figured out that in docker-compose.yaml you are setting a path to store attachments and avatar icons. ( https://github.com/wekan/wekan/blob/master/docker-compose.yml#L149 ) But it seems there no volume/pvc defined in this helm chart, is this an oversight or am I missing something?

xet7 commented 1 year ago

@Nightreaver

Yes, volume/pvc should be added with PR by someone that understands helm charts.

That WRITABLE_PATH is required to move attachments between MongoDB GridFS and filesystem. Without that setting WeKan does not work properly. More about that path is at https://github.com/wekan/wekan/pull/4336#issuecomment-1039231093

After uploading file, file is validated before moving to final storage https://github.com/wekan/wekan/pull/4648

WITH_API=true is required to have API available, and export board to JSON etc working.

I do release helm charts, because some contributors sent PRs and info about how to release them. I have not used helm charts to deploy WeKan. I have not used Kubernetes.

xet7 commented 1 year ago

Fixed at https://github.com/wekan/charts/pull/16