expunge the word 'manifest' from Kubelet's config API
The word 'manifest' technically refers to a container-group specification that predated the Pod abstraction. We should avoid using this legacy terminology where possible. Fortunately, the Kubelet's config API will be beta in 1.10 for the first time, so we still had the chance to make this change. I left the flags alone, since they're deprecated anyway. I changed a few var names in files I touched too, but this PR is the just the first shot, not the whole campaign (`git grep -i manifest | wc -l -> 1248`).
This commit is contained in:
@@ -39,16 +39,16 @@ import (
|
||||
var _ = framework.KubeDescribe("MirrorPod", func() {
|
||||
f := framework.NewDefaultFramework("mirror-pod")
|
||||
Context("when create a mirror pod ", func() {
|
||||
var ns, manifestPath, staticPodName, mirrorPodName string
|
||||
var ns, podPath, staticPodName, mirrorPodName string
|
||||
BeforeEach(func() {
|
||||
ns = f.Namespace.Name
|
||||
staticPodName = "static-pod-" + string(uuid.NewUUID())
|
||||
mirrorPodName = staticPodName + "-" + framework.TestContext.NodeName
|
||||
|
||||
manifestPath = framework.TestContext.KubeletConfig.PodManifestPath
|
||||
podPath = framework.TestContext.KubeletConfig.StaticPodPath
|
||||
|
||||
By("create the static pod")
|
||||
err := createStaticPod(manifestPath, staticPodName, ns,
|
||||
err := createStaticPod(podPath, staticPodName, ns,
|
||||
imageutils.GetE2EImage(imageutils.NginxSlim), v1.RestartPolicyAlways)
|
||||
Expect(err).ShouldNot(HaveOccurred())
|
||||
|
||||
@@ -65,7 +65,7 @@ var _ = framework.KubeDescribe("MirrorPod", func() {
|
||||
|
||||
By("update the static pod container image")
|
||||
image := imageutils.GetPauseImageNameForHostArch()
|
||||
err = createStaticPod(manifestPath, staticPodName, ns, image, v1.RestartPolicyAlways)
|
||||
err = createStaticPod(podPath, staticPodName, ns, image, v1.RestartPolicyAlways)
|
||||
Expect(err).ShouldNot(HaveOccurred())
|
||||
|
||||
By("wait for the mirror pod to be updated")
|
||||
@@ -111,7 +111,7 @@ var _ = framework.KubeDescribe("MirrorPod", func() {
|
||||
})
|
||||
AfterEach(func() {
|
||||
By("delete the static pod")
|
||||
err := deleteStaticPod(manifestPath, staticPodName, ns)
|
||||
err := deleteStaticPod(podPath, staticPodName, ns)
|
||||
Expect(err).ShouldNot(HaveOccurred())
|
||||
|
||||
By("wait for the mirror pod to disappear")
|
||||
|
Reference in New Issue
Block a user