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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to