Skip to content

fix: don't error if pod has no containerStatuses (#4471) #4473

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ianroberts wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix: don't error if pod has no containerStatuses (#4471) #4473

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4b55c8a
fix: don't error if pod has no containerStatuses (#4471)
ianroberts Mar 29, 2024
24ccc1b
Return empty list by default from get()
HomayoonAlimohammadi Mar 5, 2025
210b20e
Change back to `or []`
HomayoonAlimohammadi Mar 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion scripts/kill-host-pods.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ def post_filter_has_known_containers(pod, containers: list) -> bool:
Return true if any of the container IDs on the pod match the list of
containers passed on the second argument.
"""
for container in pod["status"]["containerStatuses"] or []:
for container in pod["status"].get("containerStatuses") or []:
try:
_, container_id = container["containerID"].split("containerd://", 2)
if container_id in containers:
Expand Down
Loading