Skip to content

Hotfix/celery memory issue#711

Closed
Prajna1999 wants to merge 6 commits intomainfrom
hotfix/celery-memory-issue
Closed

Hotfix/celery memory issue#711
Prajna1999 wants to merge 6 commits intomainfrom
hotfix/celery-memory-issue

Conversation

@Prajna1999
Copy link
Copy Markdown
Collaborator

@Prajna1999 Prajna1999 commented Mar 23, 2026

Summary

Target issue is #PLEASE_TYPE_ISSUE_NUMBER
Explain the motivation for making this change. What existing problem does the pull request solve?

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

Notes

Please add here if any other information is required for the reviewer.

Summary by CodeRabbit

  • Chores
    • Updated security configuration to exclude sensitive files from version control.
    • Modified background task initialization behavior.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 23, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1748f283-3333-44f2-9299-2e93616cf77b

📥 Commits

Reviewing files that changed from the base of the PR and between 6bb0d2c and cb6d5b7.

📒 Files selected for processing (2)
  • .gitignore
  • backend/app/celery/celery_app.py

📝 Walkthrough

Walkthrough

Updated .gitignore to exclude PEM files under backend/ directory, and commented out the celery_app.autodiscover_tasks() call in the Celery application configuration to prevent automatic task discovery at startup.

Changes

Cohort / File(s) Summary
Configuration and Ignore Rules
.gitignore
Added pattern /backend/*.pem to exclude all PEM files from version control in the backend directory.
Celery Task Discovery
backend/app/celery/celery_app.py
Disabled automatic task discovery by commenting out the celery_app.autodiscover_tasks() call at startup.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • Hotfix/celery import issues #705: Modifies backend/app/celery/celery_app.py to change Celery worker startup behavior through removal of worker warmup logic, related to task and import handling.
  • fix: import celery issues #703: Modifies backend/app/celery/celery_app.py to add worker process initialization imports, directly related to Celery startup behavior changes.

Suggested labels

bug

Suggested reviewers

  • kartpop

Poem

🐰 A rabbit's tale of tasks untold,
PEM files hidden, safe and cold,
Autodiscovery takes a rest,
Celery workers pass the test! 🌱✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/celery-memory-issue

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Prajna1999 Prajna1999 closed this Mar 23, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Ayush8923 Ayush8923 deleted the hotfix/celery-memory-issue branch April 1, 2026 08:56
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.

1 participant