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

Issue #134 Run as daemon with workers #143

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

Conversation

steverobbins
Copy link
Contributor

This is my implementation of #134.

There are 3 new commands

./scheduler_cron.sh --mode daemon --action start
./scheduler_cron.sh --mode daemon --action stop
./scheduler_cron.sh --mode daemon --action status

To keep an eye on what's going on behind the scenes, I recommend watching your process list with htop or something like

watch -n 1 "ps aux | grep scheduler | grep -v grep"  

Some of the bash requires linux (e.g. sed -r).

So far this handles generating the next schedules and running jobs when they're scheduled. What I would need help with is watchdog or any other custom Aoe_Scheduler features that I'm not very familiar with.

You can checkout my branch and test locally with

git remote add steverobbins https://github.com/steverobbins/Aoe_Scheduler.git
git remote update
git checkout steverobbins/feature/daemonize

@steverobbins steverobbins force-pushed the feature/daemonize branch 2 times, most recently from cea92b3 to a8cade6 Compare August 15, 2015 06:03
@steverobbins
Copy link
Contributor Author

Squashed and rebased on master

@steverobbins
Copy link
Contributor Author

See update. To address your questions

  • I can't think of a clever way to process always tasks. They're probably have to be run the current way like you suggested.
  • In daemonNextAction we pass the unused code so that we can search for it in the process list

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