jason810496 opened a new pull request, #60320:
URL: https://github.com/apache/airflow/pull/60320

   related: #60259
   
   ## Why
   
   Previously, we couldn't  support hot-reload for API Server  `breeze k8s dev` 
because `airflow api-sever` with `DEV_MODE=true` will try to load script source 
from `localhost:5173` (the`vite` dev server), which was also mentioned in 
https://github.com/apache/airflow/pull/59747#discussion_r2646781063.
   
   
https://github.com/apache/airflow/blob/146133348dd214c30941c68e38e1b91819d4691e/airflow-core/src/airflow/api_fastapi/core_api/app.py#L43-L75
   
   
https://github.com/apache/airflow/blob/146133348dd214c30941c68e38e1b91819d4691e/airflow-core/src/airflow/ui/dev/index.html#L1-L10
   
   ## How
   
   We can bypass the above behavior by running `airflow api-sever --dev` 
without `DEV_MODE=true` ( instead of setting `DEV_MODE=true` then run `airflow 
api-server` without `--dev` flag ), which allow us to support hot-reload for 
backend part.
   
   ## What
   
   
   We should add the `--dev` flag for API Server instead of setting 
`DEV_MODE=true` environment variable to **fully support hot-reload across all 
the Airflow components when running `breeze k8s dev` cmd** (except the for UI 
hot-reload).
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to