Merge pull request #30168 from mtaufen/test-e2e-framework-pods-name-fix

Automatic merge from submit-queue Bug fix: Use p.Name instead of pod.Name For example, if you used `pod.GenerateName`, `pod.Name` might be the empty string while `p.Name` contains the actual name of your pod. Thus passing `pod.Name` can result in a `resource name may not be empty` error.

Merge pull request #30168 from mtaufen/test-e2e-framework-pods-name-fix
ba26b227 · Kubernetes Submit Queue · GitHub · e50a491e · 73f2c55e · ba26b227
Commit ba26b227 authored Aug 05, 2016 by Kubernetes Submit Queue Committed by GitHub Aug 05, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

pods.go test/e2e/framework/pods.go +1 -1

No files found.
--- a/test/e2e/framework/pods.go
+++ b/test/e2e/framework/pods.go
@@ -56,7 +56,7 @@ func (c *PodClient) CreateSync(pod *api.Pod) *api.Pod {
 	p := c.Create(pod)
 	ExpectNoError(c.f.WaitForPodRunning(p.Name))
 	// Get the newest pod after it becomes running, some status may change after pod created, such as pod ip.
-	p, err := c.Get(pod.Name)
+	p, err := c.Get(p.Name)
 	ExpectNoError(err)
 	return p
 }