Skip to content

fix(#57): Remove unused Celery configuration and dependencies#98

Merged
YaronZaki merged 4 commits into
Quantarq:mainfrom
Amarjeet325:fix/57-remove-unused-celery
Jun 20, 2026
Merged

fix(#57): Remove unused Celery configuration and dependencies#98
YaronZaki merged 4 commits into
Quantarq:mainfrom
Amarjeet325:fix/57-remove-unused-celery

Conversation

@Amarjeet325

@Amarjeet325 Amarjeet325 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

…Quantara environment resolves #57

Problem
The docker-compose.quantara.yaml file was referencing a nonexistent quantara_tracker.celery_config module for its celery and celery_beat services. 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:

  • Removed celery and celery_beat background services from devops/docker-compose.quantara.yaml.
  • Removed the redis service and its associated volume redis_data from devops/docker-compose.quantara.yaml as it was only utilized by the removed Celery workers.
  • Removed celery and redis Python dependencies from quantara/pyproject.toml.
  • Verified that back.yaml and dev.yaml have no residual references to Celery.

Testing:

  • Verified that Docker Compose starts cleanly without attempting to launch broken background tasks.

@YaronZaki YaronZaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 YaronZaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor

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! 🎯

@Amarjeet325

Copy link
Copy Markdown
Contributor Author

Hey @YaronZaki I have resolved the conflicts; please review and merge it.

@YaronZaki YaronZaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice cleanup — pulling dead Celery infra is way better than papering over a broken config. CI is green, conflicts resolved, merging.

@YaronZaki YaronZaki merged commit c8e6f31 into Quantarq:main Jun 20, 2026
3 checks passed
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.

fix: Fix or remove Celery worker config referencing nonexistent quantara_tracker.celery_config

2 participants