pbacsko commented on code in PR #429:
URL: https://github.com/apache/yunikorn-k8shim/pull/429#discussion_r888051360


##########
test/e2e/recovery_and_restart/recovery_and_restart_test.go:
##########
@@ -128,6 +128,125 @@ var _ = ginkgo.Describe("", func() {
                
gomega.Ω(int64(resMap["vcore"].(float64))).To(gomega.Equal(core))
        })
 
+       ginkgo.It("Verify_SleepJobs_Restart_YK", func() {
+               kClient = k8s.KubeCtl{}
+               Ω(kClient.SetClient()).To(gomega.BeNil())
+               defer restorePortForwarding(&kClient)
+
+               appID1 := normalSleepJobPrefix + "-" + common.RandSeq(5)
+               sleepPodConfig1 := common.SleepPodConfig{Name: 
"normal-sleep-job", NS: dev, Time: 20, AppID: appID1}
+               pod1 := common.InitSleepPod(sleepPodConfig1)
+
+               appID2 := normalSleepJobPrefix + "-" + common.RandSeq(5)
+               sleepPodConfig2 := common.SleepPodConfig{Name: 
"normal-sleep-job-2", NS: dev, Time: 20, AppID: appID2}
+               pod2 := common.InitSleepPod(sleepPodConfig2)
+
+               ginkgo.By("Submitting two normal sleep jobs")
+               job1 := common.InitTestJob(appID1, parallelism, parallelism, 
pod1)
+               _, createErr := kClient.CreateJob(job1, dev)
+               Ω(createErr).NotTo(gomega.HaveOccurred())
+               job2 := common.InitTestJob(appID2, parallelism, parallelism, 
pod2)
+               _, createErr2 := kClient.CreateJob(job2, dev)
+               Ω(createErr2).NotTo(gomega.HaveOccurred())
+
+               ginkgo.By("Restart the scheduler pod immediately")
+               restartYunikorn(&kClient)
+
+               ginkgo.By("Listing pods")
+               pods, err := kClient.GetPods(dev)
+               Ω(err).NotTo(gomega.HaveOccurred())
+               fmt.Fprintf(ginkgo.GinkgoWriter, "Total number of pods in 
namespace %s: %d\n",
+                       dev, len(pods.Items))
+               for _, pod := range pods.Items {
+                       fmt.Fprintf(ginkgo.GinkgoWriter, "Pod name: 
%-40s\tStatus: %s\n", pod.GetName(), pod.Status.Phase)
+               }
+
+               ginkgo.By("Waiting for sleep pods to be running")

Review Comment:
   TODO: this waiting is not needed, pods will eventually terminate



##########
test/e2e/recovery_and_restart/recovery_and_restart_test.go:
##########
@@ -128,6 +128,125 @@ var _ = ginkgo.Describe("", func() {
                
gomega.Ω(int64(resMap["vcore"].(float64))).To(gomega.Equal(core))
        })
 
+       ginkgo.It("Verify_SleepJobs_Restart_YK", func() {
+               kClient = k8s.KubeCtl{}
+               Ω(kClient.SetClient()).To(gomega.BeNil())
+               defer restorePortForwarding(&kClient)
+
+               appID1 := normalSleepJobPrefix + "-" + common.RandSeq(5)
+               sleepPodConfig1 := common.SleepPodConfig{Name: 
"normal-sleep-job", NS: dev, Time: 20, AppID: appID1}
+               pod1 := common.InitSleepPod(sleepPodConfig1)
+
+               appID2 := normalSleepJobPrefix + "-" + common.RandSeq(5)
+               sleepPodConfig2 := common.SleepPodConfig{Name: 
"normal-sleep-job-2", NS: dev, Time: 20, AppID: appID2}
+               pod2 := common.InitSleepPod(sleepPodConfig2)
+
+               ginkgo.By("Submitting two normal sleep jobs")
+               job1 := common.InitTestJob(appID1, parallelism, parallelism, 
pod1)
+               _, createErr := kClient.CreateJob(job1, dev)
+               Ω(createErr).NotTo(gomega.HaveOccurred())
+               job2 := common.InitTestJob(appID2, parallelism, parallelism, 
pod2)
+               _, createErr2 := kClient.CreateJob(job2, dev)
+               Ω(createErr2).NotTo(gomega.HaveOccurred())
+
+               ginkgo.By("Restart the scheduler pod immediately")
+               restartYunikorn(&kClient)
+
+               ginkgo.By("Listing pods")
+               pods, err := kClient.GetPods(dev)
+               Ω(err).NotTo(gomega.HaveOccurred())
+               fmt.Fprintf(ginkgo.GinkgoWriter, "Total number of pods in 
namespace %s: %d\n",
+                       dev, len(pods.Items))
+               for _, pod := range pods.Items {
+                       fmt.Fprintf(ginkgo.GinkgoWriter, "Pod name: 
%-40s\tStatus: %s\n", pod.GetName(), pod.Status.Phase)
+               }
+
+               ginkgo.By("Waiting for sleep pods to be running")

Review Comment:
   TODO: this waiting is not necessary, pods will eventually terminate



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to