This is an automated email from the ASF dual-hosted git repository.

pbacsko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yunikorn-k8shim.git


The following commit(s) were added to refs/heads/master by this push:
     new 4dc1820a [YUNIKORN-3110] clean up dup error assetion in vertical pod 
scaling test (#982)
4dc1820a is described below

commit 4dc1820a35301a6ab0265644e00af287b850941e
Author: kaichiachen <[email protected]>
AuthorDate: Mon Aug 11 22:18:35 2025 +0200

    [YUNIKORN-3110] clean up dup error assetion in vertical pod scaling test 
(#982)
    
    Closes: #982
    
    Signed-off-by: Peter Bacsko <[email protected]>
---
 test/e2e/pod_resource_scaling/pod_resource_scaling_test.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/test/e2e/pod_resource_scaling/pod_resource_scaling_test.go 
b/test/e2e/pod_resource_scaling/pod_resource_scaling_test.go
index e40716ea..b447e8d8 100644
--- a/test/e2e/pod_resource_scaling/pod_resource_scaling_test.go
+++ b/test/e2e/pod_resource_scaling/pod_resource_scaling_test.go
@@ -87,7 +87,7 @@ func verifyYunikornResourceUsage(appID, resourceName string, 
value int64) {
                }
 
                return false
-       }, 30*time.Second, 120*time.Second)
+       }, time.Second, 120*time.Second)
        Ω(err).NotTo(HaveOccurred(), fmt.Sprintf("Pod should be scheduled by 
YuniKorn with correct resource(%s) allocation", resourceName))
 }
 
@@ -164,14 +164,11 @@ var _ = ginkgo.Describe("InPlacePodVerticalScaling", 
func() {
                }, time.Second, 120*time.Second)
                Ω(err).NotTo(HaveOccurred())
 
-               Ω(err).NotTo(HaveOccurred())
                Ω(pod.Status.StartTime).To(Equal(initialStartTime), "Pod should 
not have restarted")
                
Ω(pod.Status.ContainerStatuses[0].RestartCount).To(Equal(initialRestartCount), 
"Container should not have restarted")
                verifyYunikornResourceUsage(pod.Labels["applicationId"], 
"vcore", 100)
 
                pod, err = kClient.ModifyResourceUsage(pod, ns, 100, 100)
-               Ω(err).NotTo(HaveOccurred()) // Expect an error as memory 
cannot be decreased
-
                Ω(err).NotTo(HaveOccurred())
                Ω(pod.Status.StartTime).To(Equal(initialStartTime), "Pod should 
not have restarted")
                
Ω(pod.Status.ContainerStatuses[0].RestartCount).To(Equal(initialRestartCount), 
"Container should not have restarted")


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to