Worker doesn't start when minio enabled

[muratdacic]

When minio enabled, worker is searching for non existing pvc, because there is a missing condition.
Line 179 and 185 in worker-deployment.yaml, instead of

{{ if not .Values.global.s3.enabled }}

can someone just put

{{- if and (not .Values.global.s3.enabled) (not .Values.minio.enabled) }}

[grthr]

Same issue here: The PVC is still referenced in the lago-worker.

And there is no s3/minio configuration in lago-worker when minio is enabled.

[electrosenpai]

{{ if not .Values.global.s3.enabled }}

Good catch here sorry about that. PR is on the way

Should be good with #89 merge

[grthr]

what about S3 credentials in worker-deployment if using minio? I would expect S3 environment variables in the worker-deployment just like in api-deployment (lines 148-209):

lago-helm-charts/templates/api-deployment.yaml

Line 148 in 7d1856e

{{ if or .Values.global.s3.enabled .Values.minio.enabled }}

[electrosenpai]

Correct @grthr PR is on the way

[electrosenpai]

Its now merge & should be good. Feel free to re-open this issue if its doesn't solve.