Merge pull request #44727 from x1957/master

Automatic merge from submit-queue adds log when gpuManager.start() failed If gpuManager.start() returns error, there is no log. We confused with scheduler do not schedule any pod(with gpu) to one node. kubectl describe node xxx shows there is no gpu on that node, because the gpu driver do not work on that node, gpuManager.start() failed, but we can not see anything in log.

Merge pull request #44727 from x1957/master
f4fc4be8 · Kubernetes Submit Queue · GitHub · 448f8501 · 3db1127e · f4fc4be8
Commit f4fc4be8 authored May 08, 2017 by Kubernetes Submit Queue Committed by GitHub May 08, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

kubelet.go pkg/kubelet/kubelet.go +3 -1

No files found.
--- a/pkg/kubelet/kubelet.go
+++ b/pkg/kubelet/kubelet.go
@@ -1176,7 +1176,9 @@ func (kl *Kubelet) initializeModules() error {
 	}

 	// Step 7: Initialize GPUs
-	kl.gpuManager.Start()
+	if err := kl.gpuManager.Start(); err != nil {
+		glog.Errorf("Failed to start gpuManager %v", err)
+	}

 	// Step 8: Start resource analyzer
 	kl.resourceAnalyzer.Start()