-
Notifications
You must be signed in to change notification settings - Fork 2k
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
add exitcode to labels of kube_pod_init_container_status_terminated_reason #1481
Comments
I would like to work on this. I will review and update back /assign |
Would it be better to create a new metric with the exit code? Or else we would introduce a label with a large cardinality (256 possible values) |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close |
@k8s-triage-robot: Closing this issue. In response to this:
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/test-infra repository. |
/reopen Having exit code would be useful for monitoring different types of container exits. |
@prashantv: You can't reopen an issue/PR unless you authored it or you are a collaborator. In response to this:
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/test-infra repository. |
Hey @slashpai |
/reopen |
@ssabo: You can't reopen an issue/PR unless you authored it or you are a collaborator. In response to this:
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/test-infra repository. |
/reopen |
@anatoli-klamer: You can't reopen an issue/PR unless you authored it or you are a collaborator. In response to this:
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/test-infra repository. |
What would you like to be added:
Add exitcode to labels of kube_pod_init_container_status_terminated_reason
Why is this needed:
Our application has defined exit codes. We would like to be alerted when app ends up with a specified error code.
Describe the solution you'd like
We would like to define alert that uses kube_pod_init_container_status_terminated_reason metric and be alerted when container exists with certain exit code.
Additional context
We are using node.js. Node.js has limit for memory allocation. (see NODE_OPTIONS --max-old-space-size=)
We intentionally keep limit for node.js a few percent less then K8S resource limit.
In that way node.js has possibility to handle OOM (e.g. create dump file).
POD gets then signal SIGABRT 6 (exit code 128+6 = 134).
We would like to be alerted when such situation happens.
The text was updated successfully, but these errors were encountered: