[GitHub] [airflow] dimberman commented on issue #7365: [AIRFLOW-5221] add host_aliases to KubernetesPodOperator

2020-03-25 Thread GitBox
dimberman commented on issue #7365: [AIRFLOW-5221] add host_aliases to 
KubernetesPodOperator
URL: https://github.com/apache/airflow/pull/7365#issuecomment-603984814
 
 
   ```
   tests/runtime/kubernetes/test_kubernetes_executor.py .F
   
    TestKubernetesExecutor.test_integration_run_dag_with_scheduler_failure 

   
   self = 

   
   def test_integration_run_dag_with_scheduler_failure(self):
   
   host = KUBERNETES_HOST
   
   dag_id = 'example_kubernetes_executor_config'
   
   
   
   result_json = self.start_dag(dag_id=dag_id, host=host)
   
   
   
   self.assertGreater(len(result_json['items']), 0)
   
   
   
   execution_date = result_json['items'][0]['execution_date']
   
   print("Found the job with execution date {}".format(execution_date))
   
   
   
   self._delete_airflow_pod()
   
   
   
   time.sleep(10)  # give time for pod to restart
   
   
   
   # Wait some time for the operator to complete
   
   self.monitor_task(host=host,
   
 execution_date=execution_date,
   
 dag_id=dag_id,
   
 task_id='start_task',
   
 expected_final_state='success', timeout=200)
   
   
   
   self.monitor_task(host=host,
   
 execution_date=execution_date,
   
 dag_id=dag_id,
   
 task_id='other_namespace_task',
   
   > expected_final_state='success', timeout=200)
   
   tests/runtime/kubernetes/test_kubernetes_executor.py:231: 
   
   _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
_ _ 
   
   tests/runtime/kubernetes/test_kubernetes_executor.py:107: in monitor_task
   
   self.assertEqual(state, expected_final_state)
   
   E   AssertionError: '' != 'success'
   
   E   + success
   ```


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


With regards,
Apache Git Services


[GitHub] [airflow] dimberman commented on issue #7365: [AIRFLOW-5221] add host_aliases to KubernetesPodOperator

2020-03-19 Thread GitBox
dimberman commented on issue #7365: [AIRFLOW-5221] add host_aliases to 
KubernetesPodOperator
URL: https://github.com/apache/airflow/pull/7365#issuecomment-601318150
 
 
   @ashb as the config rewrite is down the road I'm good to merge this.


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


With regards,
Apache Git Services