The GitHub Actions job "Tests" on airflow.git has succeeded. Run started by GitHub user dstandish (triggered by dstandish).
Head commit for run: 3443d43f2f5e1466a1a453094da329189a064d87 / Daniel Standish <[email protected]> KubernetesHook should try incluster first when not otherwise configured Currently when K8s hook receives no configuration (e.g. incluster vs config file content vs config file path) the default client generation process will try to load the kube config in the default location. This is inconsistent with airflow core's behavior in kubernetes executor and kubernetes pod operator (in_cluster=True is the default with those). To make k8s hook's behavior consistent, we can first try incluster, then if that fails, try default kubeconfig. This should be safe to do. The kubernetes client will check for 2 environment variables that an in-cluster environment should have and if it doesn't find them, it will raise ConfigException (see here: https://github.com/kubernetes-client/python/blob/1271465acdb80bf174c50564a384fd6898635ea6/kubernetes/base/config/incluster_config.py#L60-L62). If ConfigException is raised, K8s hook will fall back to looking for the default config. Report URL: https://github.com/apache/airflow/actions/runs/2197583578 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
