[GitHub] [airflow] potiuk commented on pull request #29922: Revert "Decouple production executor warning in dags UI (#29609)"

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on PR #29922:
URL: https://github.com/apache/airflow/pull/29922#issuecomment-1454941979

   Not sure why, but I think #https://github.com/apache/airflow/pull/29609 broken pytest collection in main (looks like a circular import). For some reason it did not happen in the original PR.
   
   We should merge it (and find the cause) as both PRs started to fail with:
   
   ```
       from airflow.executors import celery_executor, celery_kubernetes_executor
   airflow/executors/celery_executor.py:40: in <module>
       from celery import Celery, Task, states as celery_states
   E   ImportError: cannot import name 'Celery' from 'celery' (/usr/local/lib/python3.7/site-packages/celery/__init__.py)
   ```
   
   For example here: https://github.com/apache/airflow/actions/runs/4333507290/jobs/7566768495


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org