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

Added standalone flag to run jobberrunner as non-root #261

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AndiDittrich
Copy link

First of all, thanks for jobber it's a great tool to handle cron tasks in a modern way!


By running jobberrunner directly within a container/docker i've experienced some problems with existing images because the directory /var/jobber/${USERID} has to be existent (as mentioned in your official dockerfiles).

Therefore i've added a standalone mode (-s flag) to jobberrunner which skips the ipc routines/initialization.

Additionally i've modified the execution order of the argument/flag processing because -v or -h won't work in case -u or -p hasn't be set.

A minimal dockerfile can now look like this:

FROM debian:latest

# user downgrade
USER www-data

# new entrypoint
ENTRYPOINT [ "/usr/local/bin/jobberrunner" , "-s", "/etc/cron.yaml" ]

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

Successfully merging this pull request may close these issues.

1 participant