Closed
Bug 1083570
Opened 10 years ago
Closed 10 years ago
Allow ES indexing to work with CELERY_ALWAYS_EAGER = True
Categories
(Marketplace Graveyard :: Code Quality, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-10-21
People
(Reporter: andy+bugzilla, Assigned: robhudson)
References
Details
(Whiteboard: [qa-])
andym robhudson: did you say you were getting
andym TypeError: run() got multiple values for keyword argument 'header'
andym i tried to just reindex apps in zamboni, they aren't showing up in ES
robhudson yep. Fix is to set celery eager to False
andym which means running rabbit mq etc right?
robhudson apparently celery with eager=True doesn't like chords. And I found a quote by Ask saying there are known bugs with eager and he'd love to remove that option
robhudson yep, that means running it through rabbitmq
andym docker doesn't have that yet
robhudson oh :(
Assignee | ||
Comment 1•10 years ago
|
||
"CELERY_ALWAYS_EAGER will never be able to emulate everything, and in the end the eager mode is of limited usefulness (and should be avoided for testing) so extending the scope of eager is of very low priority." from https://github.com/celery/celery/issues/2275
Sounds like this is a case where we may need a workaround when CELERY_ALWAYS_EAGER=True. The "fix" would then be to not use chords in this case as I believe that's the issue.
I'll add a conditional and get this working in a serialized manner.
Summary: Allow ES indexing to work with CELERY_ALWAYS_EAGER = False → Allow ES indexing to work with CELERY_ALWAYS_EAGER = True
Assignee | ||
Comment 2•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
Target Milestone: --- → 2014-10-21
You need to log in
before you can comment on or make changes to this bug.
Description
•