Fixed typo and logging

pkg/syncer/admissionhandler/validatepvc.go

@@ -18,7 +18,7 @@ import (
 
 const (
 	ExpandVolumeWithSnapshotErrorMessage = "Expanding volume with snapshots is not allowed"
-	DeleteVoluemWithSnapshotErrorMessage = "Deleting volume with snapshots is not allowed"
+	DeleteVolumeWithSnapshotErrorMessage = "Deleting volume with snapshots is not allowed"
 )
 
 // validatePVC helps validate AdmissionReview requests for PersistentVolumeClaim.
@@ -58,7 +58,7 @@ func validatePVC(ctx context.Context, ar *admissionv1.AdmissionReview) *admissio
 		oldReq := oldPVC.Spec.Resources.Requests[corev1.ResourceStorage]
 
 		if !isRWOVolumeRequest(oldPVC.Spec.AccessModes) {
-			log.Infof("the access mode of PVC is not ReadWriteOnce. skipping validation.")
+			log.Info("the access mode of PVC is not ReadWriteOnce. skipping validation.")
 			return &admissionv1.AdmissionResponse{
 				// skip validation if the pvc is not RWO
 				Allowed: true,
@@ -87,8 +87,8 @@ func validatePVC(ctx context.Context, ar *admissionv1.AdmissionReview) *admissio
 				}
 			}
 
-			if reclaimPolicy == corev1.PersistentVolumeReclaimRetain {
-				log.Infof("the reclaim policy of PVC is Retain. skipping validation.")
+			if reclaimPolicy != corev1.PersistentVolumeReclaimDelete {
+				log.Info("the reclaim policy of PVC is not Delete. skipping validation.")
 				return &admissionv1.AdmissionResponse{
 					// skip validation if the reclaim policy of PVC is not Delete
 					Allowed: true,
@@ -113,7 +113,7 @@ func validatePVC(ctx context.Context, ar *admissionv1.AdmissionReview) *admissio
 					}
 				} else if req.Operation == admissionv1.Delete {
 					result = &metav1.Status{
-						Reason: DeleteVoluemWithSnapshotErrorMessage,
+						Reason: DeleteVolumeWithSnapshotErrorMessage,
 					}
 				}
 			}

pkg/syncer/admissionhandler/validatepvc_test.go

@@ -146,7 +146,7 @@ func TestValidatePVC(t *testing.T) {
 			expectedResponse: &admissionv1.AdmissionResponse{
 				Allowed: false,
 				Result: &metav1.Status{
-					Reason: DeleteVoluemWithSnapshotErrorMessage,
+					Reason: DeleteVolumeWithSnapshotErrorMessage,
 				},
 			},
 		},