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

[Roadmap] background worker process #4304

Closed
SchrodingersGat opened this issue Feb 4, 2023 · 5 comments
Closed

[Roadmap] background worker process #4304

SchrodingersGat opened this issue Feb 4, 2023 · 5 comments
Labels
roadmap This is a roadmap feature with no immediate plans for implementation setup Relates to the InvenTree setup / installation process
Milestone

Comments

@SchrodingersGat
Copy link
Member

We have been using django-q for ages, but the project has not received any updates in a few years.

There are also some shortcomings - in particular the tasks seem to run on multiple workers - see #4290

Perhaps it is time to migrate to something like celery:

  • Better support (project is active)
  • More comprehensive feature set
  • Future proofing

I have not yet done the required reading to determine how difficult this might be, or if it would require using redis, etc.

@SchrodingersGat SchrodingersGat added roadmap This is a roadmap feature with no immediate plans for implementation setup Relates to the InvenTree setup / installation process labels Feb 4, 2023
@SchrodingersGat SchrodingersGat added this to the 1.0.0 milestone Feb 4, 2023
@matmair
Copy link
Member

matmair commented Feb 4, 2023

@SchrodingersGat the thing I like about django q is that it does not need a broker to run. To my knowledge, we will need redis to use celery. Thus making testing and small bare metal setups harder.

@SchrodingersGat
Copy link
Member Author

Indeed. I do like the simplicity of the current solution. Before switching I'd like to understand if there is a way around the current issues. The biggest one being the double execution of tasks

@SchrodingersGat
Copy link
Member Author

@matmair
Copy link
Member

matmair commented Jul 5, 2023

@SchrodingersGat can we close this now as #5167 is merged

@SchrodingersGat
Copy link
Member Author

Yes, I think we can. Sticking with django-q is more attractive now that we've found an active fork

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
roadmap This is a roadmap feature with no immediate plans for implementation setup Relates to the InvenTree setup / installation process
Projects
None yet
Development

No branches or pull requests

2 participants