diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py index 9444558399f..8bb92244a9c 100644 --- a/cinder/brick/local_dev/lvm.py +++ b/cinder/brick/local_dev/lvm.py @@ -459,7 +459,7 @@ class LVM(executor.Executor): if not lv['size'][-1].isdigit(): lvsize = lvsize[:-1] if lv['name'] == self.vg_thin_pool: - self.vg_thin_pool_size = lvsize + self.vg_thin_pool_size = float(lvsize) tpfs = self._get_thin_pool_free_space(self.vg_name, self.vg_thin_pool) self.vg_thin_pool_free_space = tpfs diff --git a/cinder/tests/unit/brick/test_brick_lvm.py b/cinder/tests/unit/brick/test_brick_lvm.py index ec1a79493f0..3a9f1d0f7ae 100644 --- a/cinder/tests/unit/brick/test_brick_lvm.py +++ b/cinder/tests/unit/brick/test_brick_lvm.py @@ -322,7 +322,7 @@ class BrickLvmTestCase(test.TestCase): self.assertEqual( "9.5g", self.vg.create_thin_pool(name=self.vg.vg_thin_pool)) - self.assertEqual("9.50", self.vg.vg_thin_pool_size) + self.assertEqual(9.50, self.vg.vg_thin_pool_size) self.assertEqual(7.6, self.vg.vg_thin_pool_free_space) self.assertEqual(3.0, self.vg.vg_provisioned_capacity) @@ -331,7 +331,7 @@ class BrickLvmTestCase(test.TestCase): self.assertEqual( "9.5g", self.vg.create_thin_pool(name=self.vg.vg_thin_pool)) - self.assertEqual("9.50", self.vg.vg_thin_pool_size) + self.assertEqual(9.50, self.vg.vg_thin_pool_size) self.assertEqual(7.6, self.vg.vg_thin_pool_free_space) self.assertEqual(3.0, self.vg.vg_provisioned_capacity)