"CronJob resource" in chart's Readme is misleading

[ibcht]

Currently, the chart's Readme describes how to set-up a "CronJob resource" to use the Nextcloud system cron instead of the default Ajax cron.

## Cronjob

This chart can utilize Kubernetes `CronJob` resource to execute [background tasks](https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/background_jobs_configuration.html).

To use this functionality, set `cronjob.enabled` parameter to `true` and switch background mode to Webcron in your nextcloud settings page.
See the [Configuration](#configuration) section for further configuration of the cronjob resource.

> **Note**: For the cronjobs to work correctly, ingress must be also enabled (set `ingress.enabled` to `true`) and `nextcloud.host` has to be publicly resolvable.

However, I find it misleading, as after enabling this feature, it turned out that no CronJob was created in my namespace. After a quick inspection, it appears that this is actually a sidecar in the main Nextcloud deployment.

Would it be possible to update the documentation in order to reflect the actual behavior ? So it is clear that this is actually a container.

Maybe I got something wrong, or there is some historical reason to this that, maybe an actual CronJob was used at one point.

Anyway, thanks again for this chart, which works as a charm 😄

[jessebot]

we had a pr for this, but it was abandoned:
https://github.com/nextcloud/helm/pull/404/files

If you'd like to submit a new PR, we can review it :)

[ibcht]

Thanks for pointing that out ! I will do that, maybe actually base the PR on the content of this one.

[ibcht]

#434 is ready for review :)

[jessebot]

this has been merged; thank you!

[ibcht]

Nice, thanks for the merge !