o-nikolas commented on code in PR #37264:
URL: https://github.com/apache/airflow/pull/37264#discussion_r1484622215
##########
airflow/www/views.py:
##########
@@ -693,6 +693,8 @@ class AirflowBaseView(BaseView):
extra_args["sqlite_warning"] = settings.engine.dialect.name == "sqlite"
if not executor.is_production:
extra_args["production_executor_warning"] = executor.__name__
+ if executor.is_healthy is False:
+ extra_args["unhealthy_executor"] = executor.__name__ or "Executor"
Review Comment:
This is getting set somewhat statically as a class attribute here. Do this
check closer to the runtime method that renders the particular page(s) you're
interested in.
--
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]
