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)

2014-Q4
x86
macOS
defect
Not set
normal

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 :(
"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
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
Target Milestone: --- → 2014-10-21
See Also: → 1083690
You need to log in before you can comment on or make changes to this bug.