Set ResourceFlavorReference as required.

kubernetes-sigs · Oct 17, 2024 · bb5673e · bb5673e
1 parent c8193b9
commit bb5673e
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/apis/kueue/v1beta1/localqueue_types.go b/apis/kueue/v1beta1/localqueue_types.go
@@ -50,6 +50,8 @@ type ClusterQueueReference string
 
 type LocalQueueFlavorStatus struct {
 	// name of the flavor.
+	// +required
+	// +kubebuilder:validation:Required
 	Name ResourceFlavorReference `json:"name"`
 
 	// resources used in the flavor.

diff --git a/keps/3122-expose-flavors-in-localqueue-status/README.md b/keps/3122-expose-flavors-in-localqueue-status/README.md
@@ -27,7 +27,7 @@
 ## Summary
 
 This KEP introduces a new status field in LocalQueue, allowing users to see
-all currently available ResourceFlavors in the LocalQueue.
+all currently available ResourceFlavors for the LocalQueue.
 
 ## Motivation
 
@@ -79,6 +79,8 @@ Create `LocalQueueFlavorStatus` API object:
 ```go
 type LocalQueueFlavorStatus struct {
   // name of the flavor.
+  // +required
+  // +kubebuilder:validation:Required
   Name ResourceFlavorReference `json:"name"`
 
   // resources used in the flavor.

diff --git a/pkg/features/kube_features.go b/pkg/features/kube_features.go
@@ -114,7 +114,7 @@ const (
 	// beta: v0.9
 	//
 	// Enable the Flavors status field in the LocalQueue, allowing users to view
-	// all currently available ResourceFlavors in the LocalQueue.
+	// all currently available ResourceFlavors for the LocalQueue.
 	ExposeFlavorsInLocalQueue featuregate.Feature = "ExposeFlavorsInLocalQueue"
 )