bucket notification - check pvcName, not pvc DFBUGS 988, 991 (#1485)

Signed-off-by: Amit Prinz Setter <[email protected]>

Author: alphaprinz

pkg/system/phase2_creating.go

@@ -1242,15 +1242,15 @@ func (r *Reconciler) ReconcileODFPersistentLoggingPVC(
 	log := r.Logger.WithField("func", "ReconcileODFPersistentLoggingPVC")
 
 	// Return if persistent logging PVC already exists
-	if pvc != nil {
+	if pvcName != nil {
 		pvc.Name = *pvcName;
 		log.Infof("PersistentLoggingPVC %s already exists and supports RWX access mode. Skipping ReconcileODFPersistentLoggingPVC.", *pvcName)
 		return nil
 	}
 
 	util.KubeCheck(pvc)
 	if pvc.UID != "" {
-		log.Infof("Persistent logging PVC %s already exists. Skipping creation.", *pvcName )
+		log.Infof("Persistent logging PVC %s already exists. Skipping creation.", pvc.Name)
 		return nil
 	}
 
@@ -1259,7 +1259,7 @@ func (r *Reconciler) ReconcileODFPersistentLoggingPVC(
 	}
 	r.Own(pvc);
 
-	log.Infof("Persistent logging PVC %s does not exist. Creating...", *pvcName)
+	log.Infof("Persistent logging PVC %s does not exist. Creating...", pvc.Name)
 	err := r.Client.Create(r.Ctx, pvc)
 	if err != nil {
 		return err