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

Reload the cron container instead of restarting it #2

Open
mrrobot47 opened this issue Jul 19, 2018 · 0 comments
Open

Reload the cron container instead of restarting it #2

mrrobot47 opened this issue Jul 19, 2018 · 0 comments
Labels
good first issue Good for newcomers

Comments

@mrrobot47
Copy link
Member

According to the current execution flow of cron-command, whenever any new cron job is added, it restarts the easyengine/cron container to accommodate the new changes. This can disrupt the cycle of already running cron-jobs.

For example, if a job is running every 10 minutes and had completed say 9 minutes before restart was executed, then it's time will start from zero again. Thus, the executing cron cycles could get a little affected by this.

A reloading signal or a HUP signal to refresh and accommodate the new changes is required to replace the current restart method.

@kirtangajjar kirtangajjar added the good first issue Good for newcomers label Sep 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants