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

Morbo + Minion workers = development joy #85

Closed
wants to merge 5 commits into from

Conversation

kiwiroy
Copy link
Contributor

@kiwiroy kiwiroy commented Aug 7, 2019

Summary

Using recurring was blocking so that neither the site, nor the minion admin interface would also respond while workers worked. Giving a subprocess to the workers affords leads to better responses.

Motivation

It is error prone and tiresome to ensure that the minion workers are using the latest code that morbo is using.

References

@kraih
Copy link
Member

kraih commented Aug 7, 2019

Not a bad approach, will need some cleaning up and better tests though.

@kraih
Copy link
Member

kraih commented Aug 12, 2019

It seems this went completely off the rails with 6967e53. Maybe this should be a plugin on CPAN. At least to figure out the correct behaviour, before thinking about how to clean up the code enough to merge it into Minion.

@kraih
Copy link
Member

kraih commented Jan 19, 2021

Thanks for your contribution, but i'm afraid this is not the right approach. And the PR has been open for a very long time without new activity.

@kraih kraih closed this Jan 19, 2021
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.

2 participants