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

Proposal: Official nextcloud cronjob container #569

Closed
rcdailey opened this issue Dec 7, 2018 · 3 comments
Closed

Proposal: Official nextcloud cronjob container #569

rcdailey opened this issue Dec 7, 2018 · 3 comments

Comments

@rcdailey
Copy link
Contributor

rcdailey commented Dec 7, 2018

I've been working on a more flexible and simpler replacement to the typical /cron.sh entrypoint task for running cron.php in Nextcloud. I'd love some feedback on what I have so far, and if the maintainers of this repo would be interested in incorporating my work. Your time and feedback is appreciated.

https://github.com/rcdailey/nextcloud-cronjob

@J0WI
Copy link
Contributor

J0WI commented Dec 10, 2018

There have been many discussions about cron in Docker. But Docker is not suitable for cronjobs and therefore we do not support a cron image.

The "Docker way" to run cronjob and IMHO the most easiest way is described here:
#361 (comment)

@rcdailey
Copy link
Contributor Author

I disagree that Docker is not suitable for cronjobs. In fact you provide busybox's crond in your official image which is contradictory to your statement.

In either case, what i have works really well and I haven't noticed any functional issues. If you guys don't want to adopt it, that's fine. I'm just offering it in case you're interested. I'll go ahead and close this issue then.

@J0WI
Copy link
Contributor

J0WI commented Dec 11, 2018

Yes, Busybox contains crond, but Docker containers are not designed to run multiple services (supervisord is a hack to workaround this).

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

No branches or pull requests

2 participants