Merge pull request #35664 from ymqytw/update_drain_test

Automatic merge from submit-queue Update drain test Update how int convert to string in the kubectl drain test. It is safer to use `strconv.Itoa()` than `string()`.

Merge pull request #35664 from ymqytw/update_drain_test
1ff0b422 · Kubernetes Submit Queue · GitHub · 8e213a4a · 08d4d8db · 1ff0b422
Commit 1ff0b422 authored Oct 30, 2016 by Kubernetes Submit Queue Committed by GitHub Oct 30, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

drain_test.go pkg/kubectl/cmd/drain_test.go +3 -2

No files found.
--- a/pkg/kubectl/cmd/drain_test.go
+++ b/pkg/kubectl/cmd/drain_test.go
@@ -26,6 +26,7 @@ import (
 	"net/url"
 	"os"
 	"reflect"
+	"strconv"
 	"strings"
 	"testing"
 	"time"
@@ -658,11 +659,11 @@ func createPods(ifCreateNewPods bool) (map[string]api.Pod, []api.Pod) {
 		if ifCreateNewPods {
 			uid = types.UID(i)
 		} else {
-			uid = types.UID(string(i) + string(i))
+			uid = types.UID(strconv.Itoa(i) + strconv.Itoa(i))
 		}
 		pod := api.Pod{
 			ObjectMeta: api.ObjectMeta{
-				Name:       "pod" + string(i),
+				Name:       "pod" + strconv.Itoa(i),
 				Namespace:  "default",
 				UID:        uid,
 				Generation: int64(i),