python icon indicating copy to clipboard operation
python copied to clipboard

Kubernetes Python Client to return "Terminating" for terminating pods

Open p172913 opened this issue 3 months ago • 14 comments

What type of PR is this? /kind bug

What this PR does / why we need it: This PR makes Kubernetes Python Client to return "Terminating" for terminating pods

Which issue(s) this PR fixes: Fixes https://github.com/kubernetes-client/python/issues/2397

Special notes for your reviewer: None

Does this PR introduce a user-facing change? None

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: None

p172913 avatar Sep 15 '25 18:09 p172913

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

k8s-ci-robot avatar Sep 15 '25 18:09 k8s-ci-robot

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: p172913 Once this PR has been reviewed and has the lgtm label, please assign roycaihw for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Sep 15 '25 18:09 k8s-ci-robot

Created new pull request for the issue #2397 . As per the request in previous pull request to rebase the commits... https://github.com/kubernetes-client/python/pull/2425

p172913 avatar Sep 15 '25 19:09 p172913

@yliaog , @roycaihw Please check and approve my PR. Thanks

p172913 avatar Sep 15 '25 19:09 p172913

@roycaihw, @yliaog Please check and approve my PR. Thanks

p172913 avatar Sep 24 '25 13:09 p172913

@roycaihw , @yliaog Please check and approve my PR. Thanks

p172913 avatar Sep 25 '25 07:09 p172913

/assign

roycaihw avatar Sep 25 '25 20:09 roycaihw

@roycaihw Is there any update on this...?

p172913 avatar Oct 22 '25 14:10 p172913

@roycaihw Is there any update on this...?

p172913 avatar Oct 27 '25 13:10 p172913

@roycaihw Is there any update on this...?

p172913 avatar Nov 05 '25 13:11 p172913

Below are valid pod phases. k8s golang client would return this set of phases, so does the this python client.

// PodPhase is a label for the condition of a pod at the current time. // +enum type PodPhase string

// These are the valid statuses of pods. const ( // PodPending means the pod has been accepted by the system, but one or more of the containers // has not been started. This includes time before being bound to a node, as well as time spent // pulling images onto the host. PodPending PodPhase = "Pending" // PodRunning means the pod has been bound to a node and all of the containers have been started. // At least one container is still running or is in the process of being restarted. PodRunning PodPhase = "Running" // PodSucceeded means that all containers in the pod have voluntarily terminated // with a container exit code of 0, and the system is not going to restart any of these containers. PodSucceeded PodPhase = "Succeeded" // PodFailed means that all containers in the pod have terminated, and at least one container has // terminated in a failure (exited with a non-zero exit code or was stopped by the system). PodFailed PodPhase = "Failed" // PodUnknown means that for some reason the state of the pod could not be obtained, typically due // to an error in communicating with the host of the pod. // Deprecated: It isn't being set since 2015 (74da3b14b0c0f658b3bb8d2def5094686d0e9095) PodUnknown PodPhase = "Unknown" )

yliaog avatar Nov 11 '25 01:11 yliaog

@roycaihw is there any update...?

p172913 avatar Nov 14 '25 15:11 p172913

@roycaihw Can you please approve this PR...

p172913 avatar Nov 26 '25 11:11 p172913

@roycaihw Can you please approve this PR...

p172913 avatar Dec 01 '25 14:12 p172913