controller: use if/else for CephFS/RBD dispatch in reconcilePV

Use if/else instead of early return to handle CephFS and RBD paths
symmetrically, improving readability.

Signed-off-by: Praveen M <m.praveen@ibm.com>

Author: iPraveenParihar

internal/controller/persistentvolume/persistentvolume.go

@@ -157,27 +157,25 @@ func (r *ReconcilePersistentVolume) reconcilePV(ctx context.Context, obj runtime
 
 			return err
 		}
+	} else {
+		rbdVolID, err := rbd.RegenerateJournal(
+			pv.Spec.CSI.VolumeAttributes,
+			pv.Spec.ClaimRef.Name,
+			volumeHandler,
+			requestName,
+			pvcNamespace,
+			r.config.ClusterName,
+			r.config.InstanceID,
+			r.config.SetMetadata,
+			cr)
+		if err != nil {
+			log.ErrorLogMsg("failed to regenerate journal %s", err)
 
-		return nil
-	}
-
-	rbdVolID, err := rbd.RegenerateJournal(
-		pv.Spec.CSI.VolumeAttributes,
-		pv.Spec.ClaimRef.Name,
-		volumeHandler,
-		requestName,
-		pvcNamespace,
-		r.config.ClusterName,
-		r.config.InstanceID,
-		r.config.SetMetadata,
-		cr)
-	if err != nil {
-		log.ErrorLogMsg("failed to regenerate journal %s", err)
-
-		return err
-	}
-	if rbdVolID != volumeHandler {
-		log.DebugLog(ctx, "volumeHandler changed from %s to %s", volumeHandler, rbdVolID)
+			return err
+		}
+		if rbdVolID != volumeHandler {
+			log.DebugLog(ctx, "volumeHandler changed from %s to %s", volumeHandler, rbdVolID)
+		}
 	}
 
 	return nil