diff --git a/pkg/kubectl/run.go b/pkg/kubectl/run.go index 554996b626e..7fe1f2fe2b4 100644 --- a/pkg/kubectl/run.go +++ b/pkg/kubectl/run.go @@ -877,8 +877,6 @@ func (BasicPod) Generate(genericParams map[string]interface{}) (runtime.Object, if len(restartPolicy) == 0 { restartPolicy = v1.RestartPolicyAlways } - // TODO: Figure out why we set ImagePullPolicy here, whether we can make it - // consistent with the other places imagePullPolicy is set using flag. pod := v1.Pod{ ObjectMeta: metav1.ObjectMeta{ Name: name, @@ -888,13 +886,12 @@ func (BasicPod) Generate(genericParams map[string]interface{}) (runtime.Object, ServiceAccountName: params["serviceaccount"], Containers: []v1.Container{ { - Name: name, - Image: params["image"], - ImagePullPolicy: v1.PullIfNotPresent, - Stdin: stdin, - StdinOnce: !leaveStdinOpen && stdin, - TTY: tty, - Resources: resourceRequirements, + Name: name, + Image: params["image"], + Stdin: stdin, + StdinOnce: !leaveStdinOpen && stdin, + TTY: tty, + Resources: resourceRequirements, }, }, DNSPolicy: v1.DNSClusterFirst, diff --git a/pkg/kubectl/run_test.go b/pkg/kubectl/run_test.go index 84aad5b3e08..0615c2ad713 100644 --- a/pkg/kubectl/run_test.go +++ b/pkg/kubectl/run_test.go @@ -442,9 +442,8 @@ func TestGeneratePod(t *testing.T) { Spec: v1.PodSpec{ Containers: []v1.Container{ { - Name: "foo", - Image: "someimage", - ImagePullPolicy: v1.PullIfNotPresent, + Name: "foo", + Image: "someimage", }, }, DNSPolicy: v1.DNSClusterFirst, @@ -514,9 +513,8 @@ func TestGeneratePod(t *testing.T) { Spec: v1.PodSpec{ Containers: []v1.Container{ { - Name: "foo", - Image: "someimage", - ImagePullPolicy: v1.PullIfNotPresent, + Name: "foo", + Image: "someimage", Ports: []v1.ContainerPort{ { ContainerPort: 80, @@ -545,9 +543,8 @@ func TestGeneratePod(t *testing.T) { Spec: v1.PodSpec{ Containers: []v1.Container{ { - Name: "foo", - Image: "someimage", - ImagePullPolicy: v1.PullIfNotPresent, + Name: "foo", + Image: "someimage", Ports: []v1.ContainerPort{ { ContainerPort: 80, @@ -587,9 +584,8 @@ func TestGeneratePod(t *testing.T) { Spec: v1.PodSpec{ Containers: []v1.Container{ { - Name: "foo", - Image: "someimage", - ImagePullPolicy: v1.PullIfNotPresent, + Name: "foo", + Image: "someimage", }, }, DNSPolicy: v1.DNSClusterFirst, @@ -614,11 +610,10 @@ func TestGeneratePod(t *testing.T) { Spec: v1.PodSpec{ Containers: []v1.Container{ { - Name: "foo", - Image: "someimage", - ImagePullPolicy: v1.PullIfNotPresent, - Stdin: true, - StdinOnce: true, + Name: "foo", + Image: "someimage", + Stdin: true, + StdinOnce: true, }, }, DNSPolicy: v1.DNSClusterFirst, @@ -644,11 +639,10 @@ func TestGeneratePod(t *testing.T) { Spec: v1.PodSpec{ Containers: []v1.Container{ { - Name: "foo", - Image: "someimage", - ImagePullPolicy: v1.PullIfNotPresent, - Stdin: true, - StdinOnce: false, + Name: "foo", + Image: "someimage", + Stdin: true, + StdinOnce: false, }, }, DNSPolicy: v1.DNSClusterFirst,