-
Notifications
You must be signed in to change notification settings - Fork 4
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
Problem: Celery worker not working #127
Comments
Ugh, that is annoying. Too bad they're not using semver properly. Proposed solution sounds good, I think it's just important to remember to update kombu when you update celery in future. |
Thanks @cole. Unfortunately, pinning kombu didn't solve the issue. We also have redis-py pinned to 2.10.6 and it seems to be some important fixes in 3.2.0. How would you feel about upgrading redis-py and use |
Just tried with that and it looks like I'm having some issues building the Docker container as I'm getting the same issue. I'll try with the kombu pinning again to see if I'm able to properly re-build the container with the new deps. |
@jraddaoui I haven't tested the kombu pinning — just saw that it is working for others in the linked issues and it sounds ok in principle. Maybe try pinning kombu and updating redis? It seems that 4.3 requires redis 3.2+ |
Okay, I got it working with both setups. I'll pin kombu until celery 4.3 is released. As @cole mentions we'll need to upgrade or remove the kombu dependency when we upgrade celery and also upgrade redis to 3.2.0 or higher. |
Sorry @cole, I missed that update. kombu 4.4 requires redis 3.2, but they don't work well with celery 4.2 and celery 4.3 is still a RC. Pinning kombu to 4.3 works in the end (it was a Docker issue on my end) and I think it's the best option until celery 4.3 is released. |
Sounds good! |
Thanks @cole! |
@stefanabreitwieser If DIP upload and descendant updates work, this can be closed. |
Sorry, the application has not been updated in the test server, I'll mark this for review once the search filters pull request is merged and deployed, which will also include the fix for this issue. |
This is now ready for review in the test instance. |
The worker crashed again with the following error:
From the Redis service logs:
This looks to be again a memory issue. I have restarted the worker for now, but we should revisit this and probably monitor the memory usage by the worker (big METS files are dumped in memory entirely in the import process), alongside Elasticsearch and Redis. |
Only Elasticsearch is currently taking 1.3 GB of memory from the 2 GB available in that instance:
Which doesn't leave a lot of space to parse big METS files at this time. |
Hi @stefanabreitwieser and @bunekcca, I guess you noted the same issue again about a couple of hours ago. I forgot that we're also running Kibana in there which, alongside Elasticsearch, were taking almost all the memory in the server, so I've increased it from 2 to 4 GB. Please, let me know how it goes. |
It's working a lot faster -- thanks Radda! |
In recent installations the Celery worker crashes with the following error:
More info:
Proposed solution is to pin
kombu==4.3.0
until Celery 4.3 gets released.The text was updated successfully, but these errors were encountered: