fix: reverted log for no config and added validation unit test

Author: atulv7

pkg/handlers/generic/mutation/autorenewcerts/inject.go

@@ -67,8 +67,12 @@ func (h *autoRenewCerts) Mutate(
 		h.variableName,
 		h.variableFieldPath...,
 	)
-	if err != nil && !variables.IsNotFoundError(err) {
-		return err
+	if err != nil {
+		if variables.IsNotFoundError(err) {
+			log.V(5).Info("Control Plane auto renew certs variable not defined")
+		} else {
+			return err
+		}
 	}
 
 	log = log.WithValues(

pkg/handlers/generic/mutation/autorenewcerts/inject_test.go

@@ -89,7 +89,7 @@ var _ = Describe("Generate auto renew patches", func() {
 		},
 		{
 			patchTest: capitest.PatchTestDef{
-				Name: "auto cert renewal set with Nutanix",
+				Name: "auto cert renewal set to 10 with Nutanix",
 				Vars: []runtimehooksv1.Variable{
 					capitest.VariableWithValue(
 						v1alpha1.ClusterConfigVariableName,
@@ -115,6 +115,31 @@ var _ = Describe("Generate auto renew patches", func() {
 				}},
 			},
 		},
+		{
+			patchTest: capitest.PatchTestDef{
+				Name: "auto cert renewal set to 0 with Nutanix",
+				Vars: []runtimehooksv1.Variable{
+					capitest.VariableWithValue(
+						v1alpha1.ClusterConfigVariableName,
+						v1alpha1.NutanixClusterConfigSpec{
+							ControlPlane: &v1alpha1.NutanixControlPlaneSpec{
+								GenericControlPlaneSpec: v1alpha1.GenericControlPlaneSpec{
+									AutoRenewCertificates: &v1alpha1.AutoRenewCertificatesSpec{
+										DaysBeforeExpiry: 0,
+									},
+								},
+							},
+						},
+					),
+				},
+				RequestItem: request.NewKubeadmControlPlaneTemplateRequestItem(""),
+				UnexpectedPatchMatchers: []capitest.JSONPatchMatcher{{
+					Operation:    "add",
+					Path:         "/spec/template/spec",
+					ValueMatcher: gomega.HaveKey("rolloutBefore"),
+				}},
+			},
+		},
 	}
 
 	// create test node for each case