use field.Required for required username

pweil- · pweil- · commit 7c1bf05acea1 · 2016-09-21T07:14:26.000-04:00
diff --git a/pkg/user/api/validation/validation.go b/pkg/user/api/validation/validation.go
@@ -155,7 +155,7 @@ func ValidateIdentity(identity *api.Identity) field.ErrorList {
 		allErrs = append(allErrs, field.Invalid(userPath.Child("name"), identity.User.Name, strings.Join(reasons, ", ")))
 	}
 	if len(identity.User.Name) == 0 && len(identity.User.UID) != 0 {
-		allErrs = append(allErrs, field.Invalid(userPath.Child("username"), "username is required when uid is provided"))
+		allErrs = append(allErrs, field.Required(userPath.Child("username"), "username is required when uid is provided"))
 	}
 	if len(identity.User.Name) != 0 && len(identity.User.UID) == 0 {
 		allErrs = append(allErrs, field.Required(userPath.Child("uid"), "uid is required when username is provided"))

Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ func ValidateIdentity(identity *api.Identity) field.ErrorList {`
`155`	`155`	`allErrs = append(allErrs, field.Invalid(userPath.Child("name"), identity.User.Name, strings.Join(reasons, ", ")))`
`156`	`156`	`}`
`157`	`157`	`if len(identity.User.Name) == 0 && len(identity.User.UID) != 0 {`
`158`		`- allErrs = append(allErrs, field.Invalid(userPath.Child("username"), "username is required when uid is provided"))`
	`158`	`+ allErrs = append(allErrs, field.Required(userPath.Child("username"), "username is required when uid is provided"))`
`159`	`159`	`}`
`160`	`160`	`if len(identity.User.Name) != 0 && len(identity.User.UID) == 0 {`
`161`	`161`	`allErrs = append(allErrs, field.Required(userPath.Child("uid"), "uid is required when username is provided"))`