dgdelahera commented on PR #23450:
URL: https://github.com/apache/airflow/pull/23450#issuecomment-1154986537

   I have found other behavior that it isn't working as expected with multiple 
containers. During the `cleanup()`, if the pod isn't in the succeeded phase, it 
will fail. But as you can see in the following pod, the `base` container is 
terminated. I will make some changes and ask for a new review.
   ```
               'container_statuses': [{'container_id': '...',
                                       'image': '...',
                                       'image_id': '...',
                                       'last_state': {'running': None,
                                                      'terminated': None,
                                                      'waiting': None},
                                       'name': 'base',
                                       'ready': False,
                                       'restart_count': 0,
                                       'started': False,
                                       'state': {'running': None,
                                                 'terminated': {'container_id': 
'...',
                                                                'exit_code': 0,
                                                                'finished_at': 
datetime.datetime(2022, 6, 14, 9, 35, 17, tzinfo=tzlocal()),
                                                                'message': None,
                                                                'reason': 
'Completed',
                                                                'signal': None,
                                                                'started_at': 
datetime.datetime(2022, 6, 14, 9, 35, 2, tzinfo=tzlocal())},
                                                 'waiting': None}},
                                      {'container_id': '...',
                                       'image': '...',
                                       'image_id': '...',
                                       'last_state': {'running': None,
                                                      'terminated': None,
                                                      'waiting': None},
                                       'name': 'envoy',
                                       'ready': True,
                                       'restart_count': 0,
                                       'started': True,
                                       'state': {'running': {'started_at': 
datetime.datetime(2022, 6, 14, 9, 35, 2, tzinfo=tzlocal())},
                                                 'terminated': None,
                                                 'waiting': None}}],
               'ephemeral_container_statuses': None,
               'host_ip': '...',
               'init_container_statuses': None,
               'message': None,
               'nominated_node_name': None,
               'phase': 'Running',
          ```


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to