Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use Kubernetes CronJob instead of sidekar with cronjob daemon #683

Open
rriemann opened this issue Jan 15, 2025 · 1 comment
Open

use Kubernetes CronJob instead of sidekar with cronjob daemon #683

rriemann opened this issue Jan 15, 2025 · 1 comment
Labels
enhancement New feature or request

Comments

@rriemann
Copy link

Description of the change

Assume in the values.yaml the cronjob is enabled:

cronjob:
  enabled: true

This requires a permanently running container as it is implemented as a container in the nextcloud deployment. One could use a K8s cronjob instead.

https://kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/

Benefits

If a K8s cronjob would be used instead, the resources would be freed when the cronjob is not running. At least this is my understanding.

Possible drawbacks

If the frequency is very high, the time of launching the container and stopping may not allow many other jobs to run in between.

@rriemann rriemann added the enhancement New feature or request label Jan 15, 2025
@rriemann rriemann changed the title Feature: A cool new feature... use Kubernetes CronJob instead of sidekar with cronjob daemon Jan 15, 2025
@provokateurin
Copy link
Member

It was like that in the past and there were problems, that's why it was changed to be a permanent container instead. I think there recently was another issue with a similar request and you should also be able to find the old issues/PRs from when this was changed.
PRs are always welcome and we try to review them in a timely manner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants