fix: add warning log when users do not set annotation

Author: devzizu

docs/design/scheduling-gates-queue-admission.md

@@ -242,12 +242,15 @@ func (alloc *Action) allocateResourcesForTasks(...) {
 
 		// Skip tasks with external (non-Volcano) scheduling gates
 		if task.SchGated {
-			// Tasks that contain the QueueAllocationGate have SchGated set to false by schedulingGateRemoval() above.
-			klog.V(4).Infof("Task %s/%s has non-Volcano gate, skipping", task.Namespace, task.Name)
+			if api.HasOnlyVolcanoSchedulingGate(task.Pod) {
+				klog.Warningf("Task %s/%s has Volcano scheduling gate but missing annotation %q, gate will not be removed automatically; add the annotation or remove the gate manually",
+					task.Namespace, task.Name, schedulingv1beta1.QueueAllocationGateKey)
+			} else {
+				klog.V(4).Infof("Task %s/%s has non-Volcano scheduling gate, skipping", task.Namespace, task.Name)
+			}
 			continue
 		}
 
-
         // ... predicate checks ...
         // PrePredicate, node filtering, prioritization:
         // If any fail, pod is marked Unschedulable (gate was already queued for removal above)

pkg/scheduler/actions/allocate/allocate.go

@@ -30,6 +30,7 @@ import (
 	"k8s.io/klog/v2"
 
 	"volcano.sh/apis/pkg/apis/scheduling"
+	schedulingv1beta1 "volcano.sh/apis/pkg/apis/scheduling/v1beta1"
 	"volcano.sh/volcano/cmd/scheduler/app/options"
 	"volcano.sh/volcano/pkg/scheduler/api"
 	"volcano.sh/volcano/pkg/scheduler/cache"
@@ -696,8 +697,12 @@ func (alloc *Action) allocateResourcesForTasks(subJob *api.SubJobInfo, tasks *ut
 
 		// Skip tasks with external (non-Volcano) scheduling gates
 		if task.SchGated {
-			// Tasks that contain the QueueAllocationGate have SchGated set to false by schedulingGateRemoval() above.
-			klog.V(4).Infof("Task %s/%s has non-Volcano gate, skipping", task.Namespace, task.Name)
+			if api.HasOnlyVolcanoSchedulingGate(task.Pod) {
+				klog.Warningf("Task %s/%s has Volcano scheduling gate but missing annotation %q, gate will not be removed automatically; add the annotation or remove the gate manually",
+					task.Namespace, task.Name, schedulingv1beta1.QueueAllocationGateKey)
+			} else {
+				klog.V(4).Infof("Task %s/%s has non-Volcano scheduling gate, skipping", task.Namespace, task.Name)
+			}
 			continue
 		}