Support preload options for shell and purge commands #8374
Support preload options for shell and purge commands #8374auvipy merged 7 commits intocelery:mainfrom
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #8374 +/- ##
==========================================
+ Coverage 87.07% 87.25% +0.17%
==========================================
Files 148 148
Lines 18491 18492 +1
Branches 3152 3152
==========================================
+ Hits 16101 16135 +34
+ Misses 2110 2068 -42
- Partials 280 289 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
|
can you fix the lint errors please https://github.com/celery/celery/actions/runs/5551154931/job/15036678192?pr=8374 ? |
Thanks. Did you have any thoughts on #8374 (comment) ? I'm not sure where there is some caching or memoization happening that is making the order of the command calls + assertions significant -- although they shouldn't be. Is there something in the Celery testing suite that can be used to guarantee a free command parser so that ini options for example are not recognized after the first preload options call ? |
Done. |
Description
Fixes #8259
The pyramid-celery package adds ini and ini-var options to the standard commands. These allow the standard Pyramid ini configuration files to config Celery. With current Celery however, only the worker, beat, and events have been able to be hooked into pyramid-celery's approach. This change supports shell and purge preload options.