fix nil pointer panic when evict

Signed-off-by: Wenbo Zhang <zhangwenbo40@huawei.com>
volcano-sh · Apr 26, 2024 · 94a8b02 · 94a8b02
1 parent ffd0588
commit 94a8b02
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/pkg/scheduler/cache/cache.go b/pkg/scheduler/cache/cache.go
@@ -862,7 +862,14 @@ func (sc *SchedulerCache) Evict(taskInfo *schedulingapi.TaskInfo, reason string)
 	}()
 
 	podgroup := &vcv1beta1.PodGroup{}
-	if err := schedulingscheme.Scheme.Convert(&job.PodGroup.PodGroup, podgroup, nil); err != nil {
+
+	if job.PodGroup == nil {
+		err = fmt.Errorf("the PodGroup of Job <%s/%s> is nil", job.Namespace, job.Name)
+	} else {
+		err = schedulingscheme.Scheme.Convert(&job.PodGroup.PodGroup, podgroup, nil)
+	}
+
+	if err != nil {
 		klog.Errorf("Error while converting PodGroup to v1alpha1.PodGroup with error: %v", err)
 		return err
 	}