Changes for Openshift - Githubissues

wekan / charts

Wekan Helm Charts to deploy to Kubernetes

MIT License

2 stars 15 forks source link

Changes for Openshift #20

Closed salleman33 closed 1 year ago

salleman33 commented 1 year ago

Hi, I propose this Pull Request to adapt the Helm chart :

to disable the initContainer with the chown command (forbidden in openshift)
disable MONGO_URL in env if we declare as a secret

Regards

xet7 commented 1 year ago

initContainer makes available shared directory correct permissions, that are required for migrating attachments. Without that there could be data loss.

Although, currently migration from year old WeKan does not work https://github.com/wekan/wekan/issues/4780#issuecomment-1626401785 , so it's for new installs only currently.

One way would be to make those file permission serttings directly to WeKan containers when building them, so that initContainer would not be needed.

I'll think.

xet7 commented 1 year ago

Hmm, because there is just openshift specific change, I presume that someone can send PR if there is any need for additional changes.