Merge pull request #51305 from hzxuzhonghu/validation-fix

Automatic merge from submit-queue (batch tested with PRs 49861, 50933, 51380, 50688, 51305) fix v1 validation return error **What this PR does / why we need it**: **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #51304 **Special notes for your reviewer**: **Release note**: ```release-note NONE ```

Merge pull request #51305 from hzxuzhonghu/validation-fix
3b2417a7 · Kubernetes Submit Queue · GitHub · 855f3e78 · 2e0156ad · 3b2417a7
Commit 3b2417a7 authored Aug 27, 2017 by Kubernetes Submit Queue Committed by GitHub Aug 27, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

validation.go pkg/api/v1/validation/validation.go +2 -2

No files found.
--- a/pkg/api/v1/validation/validation.go
+++ b/pkg/api/v1/validation/validation.go
@@ -78,7 +78,7 @@ func validateContainerResourceName(value string, fldPath *field.Path) field.Erro
 			return append(allErrs, field.Invalid(fldPath, value, "must be a standard resource for containers"))
 		}
 	}
-	return field.ErrorList{}
+	return allErrs
 }

 // ValidateResourceQuantityValue enforces that specified quantity is valid for specified resource
@@ -125,7 +125,7 @@ func validateResourceName(value string, fldPath *field.Path) field.ErrorList {
 		}
 	}

-	return field.ErrorList{}
+	return allErrs
 }

 func ValidatePodLogOptions(opts *v1.PodLogOptions) field.ErrorList {