From: Andrew Schoen <aschoen@redhat.com>
Date: Wed, 26 Jul 2017 16:26:57 +0000 (-0500)
Subject: lvm-osds: check for osd_objectstore == 'filestore'
X-Git-Tag: v3.0.0rc1~3^2~5
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=96c92a154ee578ed225714a3310ab98198528c20;p=ceph-ansible.git

lvm-osds: check for osd_objectstore == 'filestore'

ceph-volume currently only has support for filestore, not bluestore

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
---

diff --git a/roles/ceph-osd/tasks/check_mandatory_vars.yml b/roles/ceph-osd/tasks/check_mandatory_vars.yml
index 353be99ee..b62ae0633 100644
--- a/roles/ceph-osd/tasks/check_mandatory_vars.yml
+++ b/roles/ceph-osd/tasks/check_mandatory_vars.yml
@@ -56,6 +56,16 @@
     - not osd_auto_discovery
     - lvm_volumes|length == 0
 
+- name: verify osd_objectstore is 'filestore' when using lvm_osds
+  fail:
+    msg: "the lvm_osds scenario currently only works for filestore, not bluestore"
+  when:
+    - osd_group_name is defined
+    - osd_group_name in group_names
+    - lvm_osds
+    - not osd_auto_discovery
+    - osd_objectstore != 'filestore'
+
 - name: make sure the lvm_volumes variable is a dictionary
   fail:
     msg: "lvm_volumes: must be a dictionary"