Skip to content

Commit b9773b1

Browse files
committed
Handle cases where container_statuses is None
1 parent 7d06ffe commit b9773b1

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/utils/pod_manager.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
from __future__ import annotations
2020

2121
import enum
22-
import itertools
2322
import json
2423
import math
2524
import time
@@ -119,9 +118,12 @@ def get_xcom_sidecar_container_resources(self) -> str | None:
119118
def get_container_status(pod: V1Pod, container_name: str) -> V1ContainerStatus | None:
120119
"""Retrieve container status."""
121120
if pod and pod.status:
122-
container_statuses = itertools.chain(
123-
pod.status.container_statuses, pod.status.init_container_statuses
124-
)
121+
container_statuses = []
122+
if pod.status.container_statuses:
123+
container_statuses.extend(pod.status.container_statuses)
124+
if pod.status.init_container_statuses:
125+
container_statuses.extend(pod.status.init_container_statuses)
126+
125127
else:
126128
container_statuses = None
127129

providers/cncf/kubernetes/tests/unit/cncf/kubernetes/utils/test_pod_manager.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@
3838
container_is_running,
3939
container_is_succeeded,
4040
container_is_terminated,
41-
get_container_status,
4241
)
4342
from airflow.utils.timezone import utc
4443

0 commit comments

Comments
 (0)