cleanup: use err and target in recommended order to errors.Is()

The documentation has `error.Is(err, target)`, so use this as the order of the parameters. Signed-off-by: Niels de Vos <[email protected]>
ceph · Oct 11, 2024 · 112c851 · 112c851
1 parent 3802dd2
commit 112c851
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/internal/rbd/group/volume_group.go b/internal/rbd/group/volume_group.go
@@ -187,7 +187,7 @@ func (vg *volumeGroup) Delete(ctx context.Context) error {
 	}
 
 	err = librbd.GroupRemove(ioctx, name)
-	if err != nil && !errors.Is(rados.ErrNotFound, err) {
+	if err != nil && !errors.Is(err, rados.ErrNotFound) {
 		return fmt.Errorf("failed to remove volume group %q: %w", vg, err)
 	}
 
@@ -245,7 +245,7 @@ func (vg *volumeGroup) RemoveVolume(ctx context.Context, vol types.Volume) error
 
 	err := vol.RemoveFromGroup(ctx, vg)
 	if err != nil {
-		if errors.Is(librbd.ErrNotExist, err) {
+		if errors.Is(err, librbd.ErrNotExist) {
 			return nil
 		}