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
Harvest auto schedule #1114
Harvest auto schedule #1114
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One thing I might have missed: since there is a default crontab, where/how can it be overloaded?
@@ -153,28 +154,33 @@ def preview(ident): | |||
return backend.harvest() | |||
|
|||
|
|||
def schedule(ident, minute='*', hour='*', | |||
def schedule(ident, cron=None, minute='*', hour='*', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need the other karkgs? minute
, hour
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we need it, udata harvest schedule
command line use it.
Right now in this PR, you can only change it from the commandline (before you could only set it, this PRs ensure it can be changed). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👌
This PRs ensures that validated harvest sources are automatically scheduled (using the
HARVEST_DEFAULT_SCHEDULE
setting which is a cron expression) and that they can be rescheduled.