-
Notifications
You must be signed in to change notification settings - Fork 39.5k
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 an event for health check failures. #4408
Conversation
@@ -1077,6 +1077,12 @@ func (kl *Kubelet) syncPod(pod *api.BoundPod, dockerContainers dockertools.Docke | |||
containersToKeep[containerID] = empty{} | |||
continue | |||
} | |||
ref, ok := kl.getRef(containerID) | |||
if !ok { | |||
glog.Warningf("No ref for pod '%v' - '%v'", ID, name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can't possibly even compile.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I blame copy and paste. fixed.
comments addressed. ptal. |
if !ok { | ||
glog.Warningf("No ref for pod '%v' - '%v'", ID, name) | ||
} else { | ||
record.Eventf(ref, "unhealthy", "Health Check Failed %v - %v", ID, name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't use the term "Health Check". We now have both liveness and readiness probes in the API. You're reporting that the liveness probe failed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
Please see my now-hidden comment: #4408 (comment) |
678ab31
to
5e49a60
Compare
oops, comments crossed paths on the wire. addressed. |
LGTM |
Add an event for health check failures.
@brendandburns Out of an abundance of caution I reverted this change because it caused v1.3 to fail on both Shippable and Travis. Rerunning, Travis is green, but Shippable v1.3 is still failing. The error seems transient. Could you squash the commits and resubmit. |
No description provided.