938eda
From 3a42d9e2afdf04dbbfd2c507f5b2392193fda25b Mon Sep 17 00:00:00 2001
938eda
From: Vojtech Trefny <vtrefny@redhat.com>
938eda
Date: Wed, 26 May 2021 12:15:54 +0200
938eda
Subject: [PATCH] Revert "More consistent lvm errors (API break)"
938eda
938eda
This reverts commit 49ec071c6d0673224a0774d613904387c52c7381.
938eda
---
938eda
 blivet/devices/lvm.py          | 72 +++++++++++++++++-----------------
938eda
 tests/devices_test/lvm_test.py | 14 +++----
938eda
 2 files changed, 43 insertions(+), 43 deletions(-)
938eda
938eda
diff --git a/blivet/devices/lvm.py b/blivet/devices/lvm.py
938eda
index a55515fc..6d23bfba 100644
938eda
--- a/blivet/devices/lvm.py
938eda
+++ b/blivet/devices/lvm.py
938eda
@@ -307,7 +307,7 @@ class LVMVolumeGroupDevice(ContainerDevice):
938eda
     def _add_log_vol(self, lv):
938eda
         """ Add an LV to this VG. """
938eda
         if lv in self._lvs:
938eda
-            raise errors.DeviceError("lv is already part of this vg")
938eda
+            raise ValueError("lv is already part of this vg")
938eda
 
938eda
         # verify we have the space, then add it
938eda
         # do not verify for growing vg (because of ks)
938eda
@@ -340,7 +340,7 @@ class LVMVolumeGroupDevice(ContainerDevice):
938eda
     def _remove_log_vol(self, lv):
938eda
         """ Remove an LV from this VG. """
938eda
         if lv not in self.lvs:
938eda
-            raise errors.DeviceError("specified lv is not part of this vg")
938eda
+            raise ValueError("specified lv is not part of this vg")
938eda
 
938eda
         self._lvs.remove(lv)
938eda
 
938eda
@@ -415,7 +415,7 @@ class LVMVolumeGroupDevice(ContainerDevice):
938eda
     @thpool_reserve.setter
938eda
     def thpool_reserve(self, value):
938eda
         if value is not None and not isinstance(value, ThPoolReserveSpec):
938eda
-            raise AttributeError("Invalid thpool_reserve given, must be of type ThPoolReserveSpec")
938eda
+            raise ValueError("Invalid thpool_reserve given, must be of type ThPoolReserveSpec")
938eda
         self._thpool_reserve = value
938eda
 
938eda
     @property
938eda
@@ -646,14 +646,14 @@ class LVMLogicalVolumeBase(DMDevice, RaidDevice):
938eda
             if seg_type not in [None, "linear", "thin", "thin-pool", "cache", "vdo-pool", "vdo"] + lvm.raid_seg_types:
938eda
                 raise ValueError("Invalid or unsupported segment type: %s" % seg_type)
938eda
             if seg_type and seg_type in lvm.raid_seg_types and not pvs:
938eda
-                raise errors.DeviceError("List of PVs has to be given for every non-linear LV")
938eda
+                raise ValueError("List of PVs has to be given for every non-linear LV")
938eda
             elif (not seg_type or seg_type == "linear") and pvs:
938eda
                 if not all(isinstance(pv, LVPVSpec) for pv in pvs):
938eda
-                    raise errors.DeviceError("Invalid specification of PVs for a linear LV: either no or complete "
938eda
-                                             "specification (with all space split into PVs has to be given")
938eda
+                    raise ValueError("Invalid specification of PVs for a linear LV: either no or complete "
938eda
+                                     "specification (with all space split into PVs has to be given")
938eda
                 elif sum(spec.size for spec in pvs) != size:
938eda
-                    raise errors.DeviceError("Invalid specification of PVs for a linear LV: the sum of space "
938eda
-                                             "assigned to PVs is not equal to the size of the LV")
938eda
+                    raise ValueError("Invalid specification of PVs for a linear LV: the sum of space "
938eda
+                                     "assigned to PVs is not equal to the size of the LV")
938eda
 
938eda
         # When this device's format is set in the superclass constructor it will
938eda
         # try to access self.snapshots.
938eda
@@ -702,13 +702,13 @@ class LVMLogicalVolumeBase(DMDevice, RaidDevice):
938eda
         self._from_lvs = from_lvs
938eda
         if self._from_lvs:
938eda
             if exists:
938eda
-                raise errors.DeviceError("Only new LVs can be created from other LVs")
938eda
+                raise ValueError("Only new LVs can be created from other LVs")
938eda
             if size or maxsize or percent:
938eda
-                raise errors.DeviceError("Cannot specify size for a converted LV")
938eda
+                raise ValueError("Cannot specify size for a converted LV")
938eda
             if fmt:
938eda
-                raise errors.DeviceError("Cannot specify format for a converted LV")
938eda
+                raise ValueError("Cannot specify format for a converted LV")
938eda
             if any(lv.vg != self.vg for lv in self._from_lvs):
938eda
-                raise errors.DeviceError("Conversion of LVs only possible inside a VG")
938eda
+                raise ValueError("Conversion of LVs only possible inside a VG")
938eda
 
938eda
         self._cache = None
938eda
         if cache_request and not self.exists:
938eda
@@ -723,13 +723,13 @@ class LVMLogicalVolumeBase(DMDevice, RaidDevice):
938eda
             elif isinstance(pv_spec, StorageDevice):
938eda
                 self._pv_specs.append(LVPVSpec(pv_spec, Size(0)))
938eda
             else:
938eda
-                raise AttributeError("Invalid PV spec '%s' for the '%s' LV" % (pv_spec, self.name))
938eda
+                raise ValueError("Invalid PV spec '%s' for the '%s' LV" % (pv_spec, self.name))
938eda
         # Make sure any destination PVs are actually PVs in this VG
938eda
         if not set(spec.pv for spec in self._pv_specs).issubset(set(self.vg.parents)):
938eda
             missing = [r.name for r in
938eda
                        set(spec.pv for spec in self._pv_specs).difference(set(self.vg.parents))]
938eda
             msg = "invalid destination PV(s) %s for LV %s" % (missing, self.name)
938eda
-            raise errors.DeviceError(msg)
938eda
+            raise ValueError(msg)
938eda
         if self._pv_specs:
938eda
             self._assign_pv_space()
938eda
 
938eda
@@ -1072,7 +1072,7 @@ class LVMLogicalVolumeBase(DMDevice, RaidDevice):
938eda
         else:
938eda
             msg = "the specified internal LV '%s' doesn't belong to this LV ('%s')" % (int_lv.lv_name,
938eda
                                                                                        self.name)
938eda
-            raise errors.DeviceError(msg)
938eda
+            raise ValueError(msg)
938eda
 
938eda
     def populate_ksdata(self, data):
938eda
         super(LVMLogicalVolumeBase, self).populate_ksdata(data)
938eda
@@ -1171,7 +1171,7 @@ class LVMInternalLogicalVolumeMixin(object):
938eda
     def _init_check(self):
938eda
         # an internal LV should have no parents
938eda
         if self._parent_lv and self._parents:
938eda
-            raise errors.DeviceError("an internal LV should have no parents")
938eda
+            raise ValueError("an internal LV should have no parents")
938eda
 
938eda
     @property
938eda
     def is_internal_lv(self):
938eda
@@ -1231,7 +1231,7 @@ class LVMInternalLogicalVolumeMixin(object):
938eda
 
938eda
     @readonly.setter
938eda
     def readonly(self, value):  # pylint: disable=unused-argument
938eda
-        raise errors.DeviceError("Cannot make an internal LV read-write")
938eda
+        raise ValueError("Cannot make an internal LV read-write")
938eda
 
938eda
     @property
938eda
     def type(self):
938eda
@@ -1267,7 +1267,7 @@ class LVMInternalLogicalVolumeMixin(object):
938eda
     def _check_parents(self):
938eda
         # an internal LV should have no parents
938eda
         if self._parents:
938eda
-            raise errors.DeviceError("an internal LV should have no parents")
938eda
+            raise ValueError("an internal LV should have no parents")
938eda
 
938eda
     def _add_to_parents(self):
938eda
         # nothing to do here, an internal LV has no parents (in the DeviceTree's
938eda
@@ -1277,13 +1277,13 @@ class LVMInternalLogicalVolumeMixin(object):
938eda
     # internal LVs follow different rules limitting size
938eda
     def _set_size(self, newsize):
938eda
         if not isinstance(newsize, Size):
938eda
-            raise AttributeError("new size must of type Size")
938eda
+            raise ValueError("new size must of type Size")
938eda
 
938eda
         if not self.takes_extra_space:
938eda
             if newsize <= self.parent_lv.size:  # pylint: disable=no-member
938eda
                 self._size = newsize  # pylint: disable=attribute-defined-outside-init
938eda
             else:
938eda
-                raise errors.DeviceError("Internal LV cannot be bigger than its parent LV")
938eda
+                raise ValueError("Internal LV cannot be bigger than its parent LV")
938eda
         else:
938eda
             # same rules apply as for any other LV
938eda
             raise NotTypeSpecific()
938eda
@@ -1361,18 +1361,18 @@ class LVMSnapshotMixin(object):
938eda
             return
938eda
 
938eda
         if self.origin and not isinstance(self.origin, LVMLogicalVolumeDevice):
938eda
-            raise errors.DeviceError("lvm snapshot origin must be a logical volume")
938eda
+            raise ValueError("lvm snapshot origin must be a logical volume")
938eda
         if self.vorigin and not self.exists:
938eda
-            raise errors.DeviceError("only existing vorigin snapshots are supported")
938eda
+            raise ValueError("only existing vorigin snapshots are supported")
938eda
 
938eda
         if isinstance(self.origin, LVMLogicalVolumeDevice) and \
938eda
            isinstance(self.parents[0], LVMVolumeGroupDevice) and \
938eda
            self.origin.vg != self.parents[0]:
938eda
-            raise errors.DeviceError("lvm snapshot and origin must be in the same vg")
938eda
+            raise ValueError("lvm snapshot and origin must be in the same vg")
938eda
 
938eda
         if self.is_thin_lv:
938eda
             if self.origin and self.size and not self.exists:
938eda
-                raise errors.DeviceError("thin snapshot size is determined automatically")
938eda
+                raise ValueError("thin snapshot size is determined automatically")
938eda
 
938eda
     @property
938eda
     def is_snapshot_lv(self):
938eda
@@ -1544,7 +1544,7 @@ class LVMThinPoolMixin(object):
938eda
     def _check_from_lvs(self):
938eda
         if self._from_lvs:
938eda
             if len(self._from_lvs) != 2:
938eda
-                raise errors.DeviceError("two LVs required to create a thin pool")
938eda
+                raise ValueError("two LVs required to create a thin pool")
938eda
 
938eda
     def _convert_from_lvs(self):
938eda
         data_lv, metadata_lv = self._from_lvs
938eda
@@ -1590,7 +1590,7 @@ class LVMThinPoolMixin(object):
938eda
     def _add_log_vol(self, lv):
938eda
         """ Add an LV to this pool. """
938eda
         if lv in self._lvs:
938eda
-            raise errors.DeviceError("lv is already part of this vg")
938eda
+            raise ValueError("lv is already part of this vg")
938eda
 
938eda
         # TODO: add some checking to prevent overcommit for preexisting
938eda
         self.vg._add_log_vol(lv)
938eda
@@ -1601,7 +1601,7 @@ class LVMThinPoolMixin(object):
938eda
     def _remove_log_vol(self, lv):
938eda
         """ Remove an LV from this pool. """
938eda
         if lv not in self._lvs:
938eda
-            raise errors.DeviceError("specified lv is not part of this vg")
938eda
+            raise ValueError("specified lv is not part of this vg")
938eda
 
938eda
         self._lvs.remove(lv)
938eda
         self.vg._remove_log_vol(lv)
938eda
@@ -1711,14 +1711,14 @@ class LVMThinLogicalVolumeMixin(object):
938eda
         """Check that this device has parents as expected"""
938eda
         if isinstance(self.parents, (list, ParentList)):
938eda
             if len(self.parents) != 1:
938eda
-                raise errors.DeviceError("constructor requires a single thin-pool LV")
938eda
+                raise ValueError("constructor requires a single thin-pool LV")
938eda
 
938eda
             container = self.parents[0]
938eda
         else:
938eda
             container = self.parents
938eda
 
938eda
         if not container or not isinstance(container, LVMLogicalVolumeDevice) or not container.is_thin_pool:
938eda
-            raise errors.DeviceError("constructor requires a thin-pool LV")
938eda
+            raise ValueError("constructor requires a thin-pool LV")
938eda
 
938eda
     @property
938eda
     def is_thin_lv(self):
938eda
@@ -1755,7 +1755,7 @@ class LVMThinLogicalVolumeMixin(object):
938eda
 
938eda
     def _set_size(self, newsize):
938eda
         if not isinstance(newsize, Size):
938eda
-            raise AttributeError("new size must of type Size")
938eda
+            raise ValueError("new size must of type Size")
938eda
 
938eda
         newsize = self.vg.align(newsize)
938eda
         newsize = self.vg.align(util.numeric_type(newsize))
938eda
@@ -2229,7 +2229,7 @@ class LVMLogicalVolumeDevice(LVMLogicalVolumeBase, LVMInternalLogicalVolumeMixin
938eda
             container = self.parents
938eda
 
938eda
         if not isinstance(container, LVMVolumeGroupDevice):
938eda
-            raise AttributeError("constructor requires a LVMVolumeGroupDevice")
938eda
+            raise ValueError("constructor requires a LVMVolumeGroupDevice")
938eda
 
938eda
     @type_specific
938eda
     def _add_to_parents(self):
938eda
@@ -2240,12 +2240,12 @@ class LVMLogicalVolumeDevice(LVMLogicalVolumeBase, LVMInternalLogicalVolumeMixin
938eda
     @type_specific
938eda
     def _check_from_lvs(self):
938eda
         """Check the LVs to create this LV from"""
938eda
-        raise errors.DeviceError("Cannot create a new LV of type '%s' from other LVs" % self.seg_type)
938eda
+        raise ValueError("Cannot create a new LV of type '%s' from other LVs" % self.seg_type)
938eda
 
938eda
     @type_specific
938eda
     def _convert_from_lvs(self):
938eda
         """Convert the LVs to create this LV from into its internal LVs"""
938eda
-        raise errors.DeviceError("Cannot create a new LV of type '%s' from other LVs" % self.seg_type)
938eda
+        raise ValueError("Cannot create a new LV of type '%s' from other LVs" % self.seg_type)
938eda
 
938eda
     @property
938eda
     def external_dependencies(self):
938eda
@@ -2265,7 +2265,7 @@ class LVMLogicalVolumeDevice(LVMLogicalVolumeBase, LVMInternalLogicalVolumeMixin
938eda
     @type_specific
938eda
     def _set_size(self, newsize):
938eda
         if not isinstance(newsize, Size):
938eda
-            raise AttributeError("new size must be of type Size")
938eda
+            raise ValueError("new size must be of type Size")
938eda
 
938eda
         newsize = self.vg.align(newsize)
938eda
         log.debug("trying to set lv %s size to %s", self.name, newsize)
938eda
@@ -2274,7 +2274,7 @@ class LVMLogicalVolumeDevice(LVMLogicalVolumeBase, LVMInternalLogicalVolumeMixin
938eda
         # space for it. A similar reasoning applies to shrinking the LV.
938eda
         if not self.exists and newsize > self.size and newsize > self.vg.free_space + self.vg_space_used:
938eda
             log.error("failed to set size: %s short", newsize - (self.vg.free_space + self.vg_space_used))
938eda
-            raise errors.DeviceError("not enough free space in volume group")
938eda
+            raise ValueError("not enough free space in volume group")
938eda
 
938eda
         LVMLogicalVolumeBase._set_size(self, newsize)
938eda
 
938eda
@@ -2622,7 +2622,7 @@ class LVMCache(Cache):
938eda
                 spec.size = spec.pv.format.free
938eda
                 space_to_assign -= spec.pv.format.free
938eda
         if space_to_assign > 0:
938eda
-            raise errors.DeviceError("Not enough free space in the PVs for this cache: %s short" % space_to_assign)
938eda
+            raise ValueError("Not enough free space in the PVs for this cache: %s short" % space_to_assign)
938eda
 
938eda
     @property
938eda
     def size(self):
938eda
diff --git a/tests/devices_test/lvm_test.py b/tests/devices_test/lvm_test.py
938eda
index 670c91c9..4156d0bf 100644
938eda
--- a/tests/devices_test/lvm_test.py
938eda
+++ b/tests/devices_test/lvm_test.py
938eda
@@ -36,10 +36,10 @@ class LVMDeviceTest(unittest.TestCase):
938eda
         lv = LVMLogicalVolumeDevice("testlv", parents=[vg],
938eda
                                     fmt=blivet.formats.get_format("xfs"))
938eda
 
938eda
-        with six.assertRaisesRegex(self, errors.DeviceError, "lvm snapshot origin must be a logical volume"):
938eda
+        with six.assertRaisesRegex(self, ValueError, "lvm snapshot origin must be a logical volume"):
938eda
             LVMLogicalVolumeDevice("snap1", parents=[vg], origin=pv)
938eda
 
938eda
-        with six.assertRaisesRegex(self, errors.DeviceError, "only existing vorigin snapshots are supported"):
938eda
+        with six.assertRaisesRegex(self, ValueError, "only existing vorigin snapshots are supported"):
938eda
             LVMLogicalVolumeDevice("snap1", parents=[vg], vorigin=True)
938eda
 
938eda
         lv.exists = True
938eda
@@ -64,7 +64,7 @@ class LVMDeviceTest(unittest.TestCase):
938eda
         pool = LVMLogicalVolumeDevice("pool1", parents=[vg], size=Size("500 MiB"), seg_type="thin-pool")
938eda
         thinlv = LVMLogicalVolumeDevice("thinlv", parents=[pool], size=Size("200 MiB"), seg_type="thin")
938eda
 
938eda
-        with six.assertRaisesRegex(self, errors.DeviceError, "lvm snapshot origin must be a logical volume"):
938eda
+        with six.assertRaisesRegex(self, ValueError, "lvm snapshot origin must be a logical volume"):
938eda
             LVMLogicalVolumeDevice("snap1", parents=[pool], origin=pv, seg_type="thin")
938eda
 
938eda
         # now make the constructor succeed so we can test some properties
938eda
@@ -258,21 +258,21 @@ class LVMDeviceTest(unittest.TestCase):
938eda
         vg = LVMVolumeGroupDevice("testvg", parents=[pv, pv2])
938eda
 
938eda
         # pvs have to be specified for non-linear LVs
938eda
-        with self.assertRaises(errors.DeviceError):
938eda
+        with self.assertRaises(ValueError):
938eda
             lv = LVMLogicalVolumeDevice("testlv", parents=[vg], size=Size("512 MiB"),
938eda
                                         fmt=blivet.formats.get_format("xfs"),
938eda
                                         exists=False, seg_type="raid1")
938eda
-        with self.assertRaises(errors.DeviceError):
938eda
+        with self.assertRaises(ValueError):
938eda
             lv = LVMLogicalVolumeDevice("testlv", parents=[vg], size=Size("512 MiB"),
938eda
                                         fmt=blivet.formats.get_format("xfs"),
938eda
                                         exists=False, seg_type="striped")
938eda
 
938eda
         # no or complete specification has to be given for linear LVs
938eda
-        with self.assertRaises(errors.DeviceError):
938eda
+        with self.assertRaises(ValueError):
938eda
             lv = LVMLogicalVolumeDevice("testlv", parents=[vg], size=Size("512 MiB"),
938eda
                                         fmt=blivet.formats.get_format("xfs"),
938eda
                                         exists=False, pvs=[pv])
938eda
-        with self.assertRaises(errors.DeviceError):
938eda
+        with self.assertRaises(ValueError):
938eda
             pv_spec = LVPVSpec(pv, Size("256 MiB"))
938eda
             pv_spec2 = LVPVSpec(pv2, Size("250 MiB"))
938eda
             lv = LVMLogicalVolumeDevice("testlv", parents=[vg], size=Size("512 MiB"),
938eda
-- 
938eda
2.31.1
938eda