Resolved some review comments

Author: vivzbansal

pkg/apis/core/validation/validation.go

@@ -5657,23 +5657,23 @@ func ValidatePodResize(newPod, oldPod *core.Pod, opts PodValidationOptions) fiel
 				dropCPUMemoryResourcesFromContainer(&container, &oldPod.Spec.InitContainers[ix])
 				if !apiequality.Semantic.DeepEqual(container, oldPod.Spec.InitContainers[ix]) {
 					// This likely means that the user has made changes to resources other than CPU and memory for sidecar container.
-					specDiff := cmp.Diff(oldPod.Spec.InitContainers[ix], container)
-					errs := field.Forbidden(specPath, fmt.Sprintf("only cpu and memory resources for sidecar containers are mutable\n%v", specDiff))
+					errs := field.Forbidden(specPath, "only cpu and memory resources for sidecar containers are mutable")
 					allErrs = append(allErrs, errs)
-					return allErrs
 				}
 			} else if !apiequality.Semantic.DeepEqual(container, oldPod.Spec.InitContainers[ix]) { // non-restartable init container
 				// This likely means that the user has modified resources of non-sidecar init container.
-				specDiff := cmp.Diff(oldPod.Spec.InitContainers[ix], container)
-				errs := field.Forbidden(specPath, fmt.Sprintf("resources for non-sidecar init containers are immutable\n%v", specDiff))
+				errs := field.Forbidden(specPath, "resources for non-sidecar init containers are immutable")
 				allErrs = append(allErrs, errs)
-				return allErrs
 			}
 			newInitContainers = append(newInitContainers, container)
 		}
 		originalCPUMemPodSpec.InitContainers = newInitContainers
 	}
 
+	if len(allErrs) > 0 {
+		return allErrs
+	}
+
 	if !apiequality.Semantic.DeepEqual(originalCPUMemPodSpec, oldPod.Spec) {
 		// This likely means that the user has made changes to resources other than CPU and Memory.
 		specDiff := cmp.Diff(oldPod.Spec, originalCPUMemPodSpec)

pkg/kubelet/kubelet_pods_test.go

@@ -6881,6 +6881,9 @@ func TestAllocatedResourcesMatchStatus(t *testing.T) {
 
 	for _, test := range tests {
 		for _, isSidecarContainer := range []bool{false, true} {
+			if isSidecarContainer {
+				test.name += " " + "for sidecar containers"
+			}
 			t.Run(test.name, func(t *testing.T) {
 				var podStatus *kubecontainer.PodStatus
 				state := kubecontainer.ContainerStateRunning