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

Make Celery backend optional #297

Merged
merged 1 commit into from
Jun 30, 2024
Merged

Conversation

oh2fih
Copy link
Collaborator

@oh2fih oh2fih commented Jun 30, 2024

In case the celery module requirements are missing, handle the exceptions caused by the missing dependencies, and continue without the Celery backend support.

At least a partial solution for the issue #295.

- CveXplore.main - INFO     - Using mongodb as datasource, connection details: None
- CveXplore.main - INFO     - Failed to load Celery TaskHandler; continuing without the optional Celery backend.
- CveXplore.main - INFO     - Initialized CveXplore version: 0.3.33

In case the celery module requirements are missing,
handle the exceptions caused by the missing dependencies,
and continue without the Celery backend support.
@oh2fih oh2fih force-pushed the optional-celery branch from c08861d to dcf41b2 Compare June 30, 2024 06:38
@oh2fih
Copy link
Collaborator Author

oh2fih commented Jun 30, 2024

@P-T-I I just tested this together with cve-search/cve-search#1108 (and #296 is just a documentation enhancement).

  • cvesearch.db_repopulate.target i.e. cve-search/sbin/db_updater.py -v -f
  • cvesearch.db_updater.service i.e. cve-search/sbin/db_updater.py
  • cvesearch.web.service
    • GUI (frontpage, CVE pages)
    • /api/last
    • /api/cvefor/

The API is still timing out from time to time and I only get < 2 GB to the database instead of 3.5 GB, but again it's not related to any of these changes anyway. (Later tonight I'll probably get a confirmation from another instance that re-population on CVE-Search 5.0.3 with CveXplore 0.3.30 partially fails the same way.)

@P-T-I P-T-I merged commit f3ada21 into cve-search:master Jun 30, 2024
3 checks passed
@oh2fih oh2fih deleted the optional-celery branch July 1, 2024 04:39
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.

2 participants