airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] johnhofman opened a new pull request #4209: [AIRFLOW-2966] Catch ApiException in the Kubernetes Executor
Date Sun, 18 Nov 2018 21:30:38 GMT
johnhofman opened a new pull request #4209: [AIRFLOW-2966] Catch ApiException in the Kubernetes
Executor
URL: https://github.com/apache/incubator-airflow/pull/4209
 
 
   ### Description
   
   Creating a pod that exceeds a namespace's resource quota throws an ApiException. This change
catches the exception and the task is re-queued inside the Executor instead of killing the
scheduler.
   
   `click 7.0` was recently released but `flask-appbuilder 1.11.1 has requirement click==6.7`.
I have pinned `click==6.7` to make the dependencies resolve.
   
   ### Tests
   
   This adds a single test `TestKubernetesExecutor. test_run_next_exception` that covers this
single scenario. Without the changes this test fails when the ApiException is not caught.

   
   This is the first test case for the `KubernetesExecutor`,  so I needed to add the `[kubernetes]`
section to `default_test.cfg` so that the `KubernetesExecutor` can be built without exceptions.
   
   Jira ticket: https://issues.apache.org/jira/browse/AIRFLOW-2966
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

Mime
View raw message