Merge pull request #558 from duanmengkk/fix_skip

fix the skip bug
kosmos-io · May 15, 2024 · 950b120 · 950b120
1 parent 6183c40
commit 950b120
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/kubenest/controller/virtualcluster.node.controller/workflow/workflow.go b/pkg/kubenest/controller/virtualcluster.node.controller/workflow/workflow.go
@@ -47,14 +47,14 @@ func (w WorkflowData) RunTask(ctx context.Context, opt task.TaskOpt) error {
 	var args interface{}
 	for i, t := range w.Tasks {
 		klog.V(4).Infof("HHHHHHHHHHHH (%d/%d) work flow run t %s  HHHHHHHHHHHH", i+1, len(w.Tasks), t.Name)
-		if t.Skip(ctx, opt) {
+		if t.Skip != nil && t.Skip(ctx, opt) {
 			klog.V(4).Infof("work flow skip task %s", t.Name)
 			continue
 		}
 		if len(t.SubTasks) > 0 {
 			for j, subTask := range t.SubTasks {
 				klog.V(4).Infof("HHHHHHHHHHHH (%d/%d) work flow run sub t %s HHHHHHHHHHHH", j+1, len(t.SubTasks), subTask.Name)
-				if t.Skip(ctx, opt) {
+				if t.Skip != nil && t.Skip(ctx, opt) {
 					klog.V(4).Infof("work flow skip sub task %s", t.Name)
 					continue
 				}