Uh oh!
There was an error while loading. Please reload this page.
Remove global from celery provider - #58869
Conversation
Uh oh!
There was an error while loading. Please reload this page.
7d647ea to
9a1f29dCompare9a1f29d to
e326e6dCompareUh oh!
There was an error while loading. Please reload this page.
* Remove global from celery provider * Remove global from celery provider, add caching
* Remove global from celery provider * Remove global from celery provider, add caching
We're pretty sure that we traced worker processes getting We upgraded Airflow 3.1.1 → 3.1.6 and while doing so went By downgrading UPDATE: Looks like we got confused during debugging and it was actually the change 3.13.1 → 3.14.0 from #58591. |
jscheffl
commented
Jan 29, 2026
@michaelosthege hypothesis ... if not duplicate hostname could it be that the duplicate hostname check sometimes needs longer running into timeout as well? Not sure about a specific error message... |
michaelosthege
commented
Jan 29, 2026
No idea how long that code takes to execute, but we had tried increasing How does the UP message travel to the parent? Via the message broker? We also observed a "clocks out of sync" warning in the logs, but with the worker and Redis containers running on the same host that doesn't make any sense. |
* Remove global from celery provider * Remove global from celery provider, add caching
Another small increment to remove global statements for PR #58116
This removes global statement in celery, a classsic thing where
functools.cache()can help. Also centralization of config retrieval in this case makes code better in general.globalis evil.