Merge pull request #155 from nbalacha/fix-1

fix: ignore LVMVolumeGroup NotFound error
openshift · Apr 18, 2022 · bdb698d · bdb698d
2 parents 8ecfd77 + 650fab1
commit bdb698d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pkg/vgmanager/vgmanager_controller.go b/pkg/vgmanager/vgmanager_controller.go
@@ -74,10 +74,10 @@ func (r *VGReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Re
 	err := r.Client.Get(ctx, req.NamespacedName, volumeGroup)
 	if err != nil {
 		r.Log.Error(err, "failed to get LVMVolumeGroup", "VGName", req.Name)
-		if !errors.IsNotFound(err) {
-			return reconcileAgain, err
+		if errors.IsNotFound(err) {
+			return ctrl.Result{}, nil
 		}
-		return ctrl.Result{}, err
+		return reconcileAgain, err
 	}
 	// Check if the nodeSelector matches the labels on this node
 	nodeMatches, err := r.matchesThisNode(ctx, volumeGroup.Spec.NodeSelector)