From 71fcd35c3d44795f92b17e9aedb7ca868b3750d3 Mon Sep 17 00:00:00 2001 From: Alfredo Deza Date: Fri, 7 Sep 2018 14:04:58 -0400 Subject: [PATCH] ceph-volume lvm.batch.filestore capture SizeAllocationErrors Signed-off-by: Alfredo Deza --- .../ceph_volume/devices/lvm/strategies/filestore.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/ceph-volume/ceph_volume/devices/lvm/strategies/filestore.py b/src/ceph-volume/ceph_volume/devices/lvm/strategies/filestore.py index 9e80c5cbb90..6cf9fd9a14d 100644 --- a/src/ceph-volume/ceph_volume/devices/lvm/strategies/filestore.py +++ b/src/ceph-volume/ceph_volume/devices/lvm/strategies/filestore.py @@ -5,6 +5,7 @@ from ceph_volume.api import lvm from . import validators from ceph_volume.devices.lvm.create import Create from ceph_volume.util import templates +from ceph_volume.exceptions import SizeAllocationError class SingleType(object): @@ -226,9 +227,7 @@ class MixedType(object): self.vg_extents = lvm.sizing( self.total_available_journal_space.b, size=self.journal_size.b ) - # FIXME with real exception catching from sizing that happens when the - # journal space is not enough - except Exception: + except SizeAllocationError: self.vg_extents = {'parts': 0, 'percentages': 0, 'sizes': 0} # validate that number of journals possible are enough for number of -- 2.39.5