Merge pull request #119522 from YTGhost/validation-for-static-pods-name

add validation for static pods to have a name
This commit is contained in:
Kubernetes Prow Robot 2023-10-17 03:38:15 +02:00 committed by GitHub
commit 873eb57a4e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -120,6 +120,10 @@ func tryDecodeSinglePod(data []byte, defaultFn defaultFunc) (parsed bool, pod *v
return false, pod, fmt.Errorf("invalid pod: %#v", obj)
}
if newPod.Name == "" {
return true, pod, fmt.Errorf("invalid pod: name is needed for the pod")
}
// Apply default values and validate the pod.
if err = defaultFn(newPod); err != nil {
return true, pod, err
@ -151,6 +155,9 @@ func tryDecodePodList(data []byte, defaultFn defaultFunc) (parsed bool, pods v1.
// Apply default values and validate pods.
for i := range newPods.Items {
newPod := &newPods.Items[i]
if newPod.Name == "" {
return true, pods, fmt.Errorf("invalid pod: name is needed for the pod")
}
if err = defaultFn(newPod); err != nil {
return true, pods, err
}