nextcloud / helm

A community maintained helm chart for deploying Nextcloud on Kubernetes.
GNU Affero General Public License v3.0
325 stars 264 forks source link

[cron] run a system cron job inside the nextcloud image #461

Closed MichaelSp closed 11 months ago

MichaelSp commented 11 months ago

Pull Request

Description of the change

Instead of running the cron as a sidecar, with this change the cron can run inside the main container by setting cronjob.asSidecar=false. Default will stay true for backwards compatability.

Benefits

run cron inside main container

Possible drawbacks

none. Backwards comparability maintained

Applicable issues

Additional information

Checklist

provokateurin commented 11 months ago

Why do you want to have this? I don't see any benefit in this other than that it could break when messing with the command line.

MichaelSp commented 11 months ago

hi @provokateurin

I tried to work around a problem I had with the sidecar cron. Turns out the cron works just fine. Can't recall what the issue was. Looks like we can close this.