Better message for RayCluster enableIngress field validation

astefanutti · openshift-merge-bot[bot] · commit 7c644080f71e · 2024-04-18T14:38:45.000Z
diff --git a/pkg/controllers/raycluster_webhook.go b/pkg/controllers/raycluster_webhook.go
@@ -147,7 +147,7 @@ func validateIngress(rayCluster *rayv1.RayCluster) field.ErrorList {
 		allErrors = append(allErrors, field.Invalid(
 			field.NewPath("spec", "headGroupSpec", "enableIngress"),
 			rayCluster.Spec.HeadGroupSpec.EnableIngress,
-			"RayCluster resources with EnableIngress set to true or unspecified is not allowed"))
+			"RayCluster with enableIngress set to true is not allowed"))
 	}
 
 	return allErrors

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ func validateIngress(rayCluster *rayv1.RayCluster) field.ErrorList {`
`147`	`147`	`allErrors = append(allErrors, field.Invalid(`
`148`	`148`	`field.NewPath("spec", "headGroupSpec", "enableIngress"),`
`149`	`149`	`rayCluster.Spec.HeadGroupSpec.EnableIngress,`
`150`		`- "RayCluster resources with EnableIngress set to true or unspecified is not allowed"))`
	`150`	`+ "RayCluster with enableIngress set to true is not allowed"))`
`151`	`151`	`}`
`152`	`152`
`153`	`153`	`return allErrors`