Skip to content
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.

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

More comprehensive reporting of termination reasons #137

Closed
bgrant0607 opened this issue Jun 17, 2014 · 2 comments
Closed

More comprehensive reporting of termination reasons #137

bgrant0607 opened this issue Jun 17, 2014 · 2 comments
Assignees
Labels
area/docker area/introspection area/kubelet kind/bug Categorizes issue or PR as related to a bug. sig/node Categorizes an issue or PR as relevant to SIG Node.

Comments

@bgrant0607
Copy link
Member

Docker reports State.Running, State.Paused, State.ExitCode (negative for signals IIRC), and State.FinishedAt. It would be useful to collect even more termination reason information, such as:

  • stop/restart operations, with client-provided reason, such as "cancel", "reload", "resize", "move", "host_update"
  • liveness probe failures (see Add extended health checking of pods/containers #66)
  • other identifiable failure reasons: OOM, container creation error, docker crash, machine crash, reboot, lost/ghost
@bgrant0607
Copy link
Member Author

We also need to think about how to report this information to clients if pods are deleted upon termination. Clients will want to watch for termination and receive the reason and other details. Logs are not sufficient for programmatic use.

@dchen1107
Copy link
Member

#1569 was merged. All available termination reasons are propagated to apiserver through ContainerStatus. Several docker issues were filed, and some of them already handled by docker. Several ones including oom are in @rjnagal's hand already. Once those docker issues are addressed properly, the related reasons are going to propagated automatically without further changes.

Client initialized reason will be handled separately. Close this one.

@dchen1107 dchen1107 added the sig/node Categorizes an issue or PR as relevant to SIG Node. label Feb 4, 2015
resouer pushed a commit to resouer/kubernetes that referenced this issue Sep 15, 2016
lazypower pushed a commit to lazypower/kubernetes that referenced this issue Jun 5, 2017
krunalhinguu pushed a commit to krunalhinguu/kubernetes that referenced this issue Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docker area/introspection area/kubelet kind/bug Categorizes issue or PR as related to a bug. sig/node Categorizes an issue or PR as relevant to SIG Node.
Projects
None yet
Development

No branches or pull requests

4 participants