pod could be preempted by default

Signed-off-by: Zhe Jin <jinzhe.hit@gmail.com>
volcano-sh · Nov 30, 2022 · 68d02c8 · 68d02c8
1 parent b2b4577
commit 68d02c8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/pkg/scheduler/api/pod_info.go b/pkg/scheduler/api/pod_info.go
@@ -98,7 +98,7 @@ func GetPodPreemptable(pod *v1.Pod) bool {
 		}
 	}
 
-	return false
+	return true
 }
 
 // GetPodRevocableZone return volcano.sh/revocable-zone value for pod/podgroup

diff --git a/pkg/scheduler/plugins/tdm/tdm_test.go b/pkg/scheduler/plugins/tdm/tdm_test.go
@@ -330,6 +330,9 @@ func Test_TDM_victimsFn(t *testing.T) {
 	p2.Annotations[schedulingv2.PodPreemptable] = "true"
 	p3.Annotations[schedulingv2.PodPreemptable] = "true"
 
+	p4.Annotations[schedulingv2.PodPreemptable] = "false"
+	p5.Annotations[schedulingv2.PodPreemptable] = "false"
+
 	p6.Annotations[schedulingv2.PodPreemptable] = "true"
 	p7.Annotations[schedulingv2.PodPreemptable] = "true"
 	p8.Annotations[schedulingv2.PodPreemptable] = "true"