Merge pull request #3769 from luthermonson/fix-nil-pointer-panic

More nil checks for Version

Author: k8s-ci-robot

controlplane/eks/api/v1beta2/awsmanagedcontrolplane_webhook.go

@@ -213,7 +213,7 @@ func (r *AWSManagedControlPlane) validateEKSVersion(old *AWSManagedControlPlane)
 		allErrs = append(allErrs, field.Invalid(path, *r.Spec.Version, err.Error()))
 	}
 
-	if old != nil {
+	if old != nil && old.Spec.Version != nil {
 		oldV, err := parseEKSVersion(*old.Spec.Version)
 		if err == nil && (v.Major() < oldV.Major() || v.Minor() < oldV.Minor()) {
 			allErrs = append(allErrs, field.Invalid(path, *r.Spec.Version, "new version less than old version"))
@@ -236,6 +236,10 @@ func (r *AWSManagedControlPlane) validateEKSAddons() field.ErrorList {
 		return allErrs
 	}
 
+	if r.Spec.Version == nil {
+		return allErrs
+	}
+
 	path := field.NewPath("spec.version")
 	v, err := parseEKSVersion(*r.Spec.Version)
 	if err != nil {