OmerJog commented on issue #5813: [AIRFLOW-4526] KubernetesPodOperator gets 
stuck in Running state when get_logs is set to True and there is a long gap 
without logs from pod
URL: https://github.com/apache/airflow/pull/5813#issuecomment-543019794
 
 
   @thealmightygrant The tests are failing with:
   ```
   52) ERROR: test_read_pod_logs_retries_fails 
(tests.kubernetes.test_pod_launcher.TestPodLauncher)
   ----------------------------------------------------------------------
      Traceback (most recent call last):
       tests/kubernetes/test_pod_launcher.py line 75 in 
test_read_pod_logs_retries_fails
         mock.sentinel
       /usr/local/lib/python3.6/site-packages/tenacity/__init__.py line 241 in 
wrapped_f
         return self.call(f, *args, **kw)
       /usr/local/lib/python3.6/site-packages/tenacity/__init__.py line 330 in 
call
         start_time=start_time)
       /usr/local/lib/python3.6/site-packages/tenacity/__init__.py line 297 in 
iter
         raise retry_exc.reraise()
       /usr/local/lib/python3.6/site-packages/tenacity/__init__.py line 136 in 
reraise
         raise self.last_attempt.result()
       /usr/local/lib/python3.6/concurrent/futures/_base.py line 425 in result
         return self.__get_result()
       /usr/local/lib/python3.6/concurrent/futures/_base.py line 384 in 
__get_result
         raise self._exception
       /usr/local/lib/python3.6/site-packages/tenacity/__init__.py line 333 in 
call
         result = fn(*args, **kwargs)
      TypeError: read_pod_logs() missing 1 required positional argument: 
'logs_connection_timeout'
   ```

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

Reply via email to