Merge pull request #37680 from mfojtik/report-pod-name

Automatic merge from submit-queue Report the Pod name and namespace when kubelet fails to sync the container This helps debugging problems with SELinux (and other problems related to the Docker failed to run the container) as currently only the UUID of the Pod is reported: ``` Error syncing pod 670f607d-b5a8-11a4-b673-005056b7468b, skipping: failed to "StartContainer" for "deployment" with RunContainerError: "runContainer: Error response from daemon: Relabeling content in /usr is not allowed." ``` Here it would be useful to know what pod in which namespace is trying to mount the "/usr".

Merge pull request #37680 from mfojtik/report-pod-name
d270ec0e · Kubernetes Submit Queue · GitHub · e1900f70 · 34db84ef · d270ec0e
Commit d270ec0e authored Jan 16, 2017 by Kubernetes Submit Queue Committed by GitHub Jan 16, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

pod_workers.go pkg/kubelet/pod_workers.go +2 -1

No files found.
--- a/pkg/kubelet/pod_workers.go
+++ b/pkg/kubelet/pod_workers.go
@@ -31,6 +31,7 @@ import (
 	"k8s.io/kubernetes/pkg/kubelet/events"
 	"k8s.io/kubernetes/pkg/kubelet/eviction"
 	kubetypes "k8s.io/kubernetes/pkg/kubelet/types"
+	"k8s.io/kubernetes/pkg/kubelet/util/format"
 	"k8s.io/kubernetes/pkg/kubelet/util/queue"
 )
@@ -178,7 +179,7 @@ func (p *podWorkers) managePodLoop(podUpdates <-chan UpdatePodOptions) {
 			update.OnCompleteFunc(err)
 		}
 		if err != nil {
-			glog.Errorf("Error syncing pod %s, skipping: %v", update.Pod.UID, err)
+			glog.Errorf("Error syncing pod %s (%q), skipping: %v", update.Pod.UID, format.Pod(update.Pod), err)
 			p.recorder.Eventf(update.Pod, v1.EventTypeWarning, events.FailedSync, "Error syncing pod, skipping: %v", err)
 		}
 		p.wrapUp(update.Pod.UID, err)