Rename parent-uid label to job-uid, change label validation

pkg/controller/constants/constants.go

@@ -33,7 +33,7 @@ const (
 	// status based on the admission status of the parent workload.
 	ParentWorkloadAnnotation = "kueue.x-k8s.io/parent-workload"
 
-	// ParentUIDLabel is the label key in the workload resource, that holds the UID of
-	// a parent.
-	ParentUIDLabel = "kueue.x-k8s.io/parent-uid"
+	// JobUIDLabel is the label key in the workload resource, that holds the UID of
+	// a parent job.
+	JobUIDLabel = "kueue.x-k8s.io/job-uid"
 )

pkg/controller/jobframework/reconciler.go

@@ -17,7 +17,6 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"unicode/utf8"
 
 	corev1 "k8s.io/api/core/v1"
 	apierrors "k8s.io/apimachinery/pkg/api/errors"
@@ -471,8 +470,8 @@ func (r *JobReconciler) constructWorkload(ctx context.Context, job GenericJob, o
 		},
 	}
 
-	if uid := string(job.Object().GetUID()); utf8.RuneCountInString(uid) < 64 && uid != "" {
-		wl.Labels[controllerconsts.ParentUIDLabel] = uid
+	if uid := string(job.Object().GetUID()); len(uid) < 64 && uid != "" {
+		wl.Labels[controllerconsts.JobUIDLabel] = uid
 	}
 
 	priorityClassName, p, err := r.extractPriority(ctx, podSets, job)

pkg/controller/jobs/job/job_controller_test.go

@@ -442,7 +442,7 @@ func TestReconciler(t *testing.T) {
 					Queue("test-queue").
 					Priority(0).
 					SetLabels(map[string]string{
-						controllerconsts.ParentUIDLabel: "test-uid",
+						controllerconsts.JobUIDLabel: "test-uid",
 					}).
 					Obj(),
 			},