From: Alfredo Deza <adeza@redhat.com>
Date: Fri, 7 Sep 2018 18:04:58 +0000 (-0400)
Subject: ceph-volume lvm.batch.filestore capture SizeAllocationErrors
X-Git-Tag: v14.0.1~344^2~7
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=71fcd35c3d44795f92b17e9aedb7ca868b3750d3;p=ceph-ci.git

ceph-volume lvm.batch.filestore capture SizeAllocationErrors

Signed-off-by: Alfredo Deza <adeza@redhat.com>
---

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