fix(#57): Remove unused Celery configuration and dependencies#98
fix(#57): Remove unused Celery configuration and dependencies#98Amarjeet325 wants to merge 4 commits into
Conversation
…Quantara environment
…and configure project dependencies
YaronZaki
left a comment
There was a problem hiding this comment.
Good cleanup — since Celery isn't being used, removing it (and the dead docker-compose services and pyproject deps) is cleaner than patching the broken config. Approving and merging.
YaronZaki
left a comment
There was a problem hiding this comment.
Heads up — the recent merges into main have introduced a small conflict in your branch (quantara/frontend/package.json + yarn.lock). The Celery removal itself in the Python side is fine, no conflict there. Could you rebase on main and push? I’ll merge right after.
|
Hey @Amarjeet325 👋 Love the cleanup! Removing the dead Celery/Redis config is exactly what #57 was asking for, and pruning the lock files alongside it is the right call. The branch has a merge conflict against main now though — could you rebase and update? Already approved in principle, just needs a clean merge. Thanks! 🎯 |
|
Hey @YaronZaki I have resolved the conflicts; please review and merge it. |
…Quantara environment resolves #57
Problem
The
docker-compose.quantara.yamlfile was referencing a nonexistentquantara_tracker.celery_configmodule for itsceleryandcelery_beatservices. This caused the background workers to fail on startup. Since a codebase audit confirmed that Celery tasks are not actively used anywhere in the application, the missing configuration is not needed.Solution
Opted to cleanly remove all unused Celery infrastructure rather than adding dummy configuration modules.
Changes Made:
celeryandcelery_beatbackground services fromdevops/docker-compose.quantara.yaml.redisservice and its associated volumeredis_datafromdevops/docker-compose.quantara.yamlas it was only utilized by the removed Celery workers.celeryandredisPython dependencies fromquantara/pyproject.toml.back.yamlanddev.yamlhave no residual references to Celery.Testing: