You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/11/05 13:10:53 UTC

[GitHub] [airflow] mik-laj commented on a change in pull request #11815: Render k8s yaml for tasks via the Airflow UI

mik-laj commented on a change in pull request #11815:
URL: https://github.com/apache/airflow/pull/11815#discussion_r518038216



##########
File path: airflow/models/taskinstance.py
##########
@@ -70,6 +70,15 @@
 from airflow.utils.state import State
 from airflow.utils.timeout import timeout
 
+try:
+    from kubernetes.client.api_client import ApiClient
+
+    from airflow.executors.kubernetes_executor import KubeConfig, create_pod_id
+    from airflow.kubernetes.pod_generator import PodGenerator
+    from airflow.settings import pod_mutation_hook

Review comment:
       No. It alway, bu we should use`from airflow import settings` to avoid cache. 




----------------------------------------------------------------
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.

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