Merge pull request #5853 from fgrzadkowski/better_errors

Improve error message for invalid pod spec update
This commit is contained in:
Wojciech Tyczynski 2015-03-25 14:31:13 +01:00
commit c8e89bc717

View File

@ -701,8 +701,7 @@ func ValidatePodUpdate(newPod, oldPod *api.Pod) errs.ValidationErrorList {
} }
pod.Spec.Containers = newContainers pod.Spec.Containers = newContainers
if !api.Semantic.DeepEqual(pod.Spec, oldPod.Spec) { if !api.Semantic.DeepEqual(pod.Spec, oldPod.Spec) {
// TODO: a better error would include all immutable fields explicitly. allErrs = append(allErrs, errs.NewFieldInvalid("spec", newPod.Spec, "may not update fields other than container.image"))
allErrs = append(allErrs, errs.NewFieldInvalid("spec.containers", newPod.Spec.Containers, "some fields are immutable"))
} }
newPod.Status = oldPod.Status newPod.Status = oldPod.Status