fix: also retry client errors

Signed-off-by: Jakob Möller <jmoller@redhat.com>
topolvm · Apr 26, 2024 · bcb92e5 · bcb92e5
1 parent be92bdb
commit bcb92e5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/driver/internal/k8s/logicalvolume_service.go b/internal/driver/internal/k8s/logicalvolume_service.go
@@ -253,7 +253,7 @@ func (s *LogicalVolumeService) DeleteVolume(ctx context.Context, volumeID string
 					return true, nil
 				}
 				logger.Error(err, "failed to get LogicalVolume", "name", lv.Name)
-				return true, err
+				return false, nil
 			}
 			logger.Info("waiting for LogicalVolume to be deleted", "name", lv.Name)
 			return false, nil
@@ -325,7 +325,7 @@ func (s *LogicalVolumeService) ExpandVolume(ctx context.Context, volumeID string
 		var changedLV topolvmv1.LogicalVolume
 		if err := s.getter.Get(ctx, client.ObjectKey{Name: lv.Name}, &changedLV); err != nil {
 			logger.Error(err, "failed to get LogicalVolume", "name", lv.Name)
-			return true, err
+			return false, nil
 		}
 
 		if changedLV.Status.Code != codes.OK {
@@ -377,7 +377,7 @@ func (s *LogicalVolumeService) updateSpecSize(ctx context.Context, volumeID stri
 					return false, nil
 				} else {
 					logger.Error(err, "failed to update LogicalVolume spec", "name", lv.Name)
-					return true, err
+					return false, nil
 				}
 			}
 			return true, nil