add recurring task support for katello#299
add recurring task support for katello#299archanaserver wants to merge 1 commit intotheforeman:masterfrom
Conversation
ekohl
left a comment
There was a problem hiding this comment.
Is there an issue somewhere to rewrite it into Foreman's own recurring logic? Katello already depends on it so I'd think it could be done.
This is an architectural choice that I think we can drive. The foreman-tasks cleanup may be an exception because there is a bigger risk (cleaning up a task from a task can break if you aren't careful) but Katello can rely on it.
If that isn't too hard for the Katello devs to do then we'll have a better and more scalable design.
|
Let's ask @ianballou |
|
Here is the issue: https://projects.theforeman.org/issues/35237 |
|
needs a rebase |
459c2d0 to
e1b9f1f
Compare
ekohl
left a comment
There was a problem hiding this comment.
Should we wait for Katello/katello#11633 to be merged and never implement the old way?
Yes, I think doing everything via Foreman::Cron is the correct way forward |
based on #288