From: Andrew Schoen Date: Thu, 10 Jan 2019 17:22:53 +0000 (-0600) Subject: validate: do not validate lvm config if osd_auto_discovery is true X-Git-Tag: v4.0.0beta1~25 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=c5b082848fa3bb1545af5371d631ce983ea81210;p=ceph-ansible.git validate: do not validate lvm config if osd_auto_discovery is true If osd_auto_discovery is set with the lvm scenario it's expected for lvm_volumes and devices to be empty. Signed-off-by: Andrew Schoen --- diff --git a/plugins/actions/validate.py b/plugins/actions/validate.py index e32c926ab..5d4a12505 100644 --- a/plugins/actions/validate.py +++ b/plugins/actions/validate.py @@ -90,12 +90,13 @@ class ActionModule(ActionBase): notario.validate(host_vars, non_collocated_osd_scenario, defined_keys=True) if host_vars["osd_scenario"] == "lvm": - if host_vars.get("devices"): - notario.validate(host_vars, lvm_batch_scenario, defined_keys=True) - elif notario_store['osd_objectstore'] == 'filestore': - notario.validate(host_vars, lvm_filestore_scenario, defined_keys=True) - elif notario_store['osd_objectstore'] == 'bluestore': - notario.validate(host_vars, lvm_bluestore_scenario, defined_keys=True) + if not host_vars.get("osd_auto_discovery", False): + if host_vars.get("devices"): + notario.validate(host_vars, lvm_batch_scenario, defined_keys=True) + elif notario_store['osd_objectstore'] == 'filestore': + notario.validate(host_vars, lvm_filestore_scenario, defined_keys=True) + elif notario_store['osd_objectstore'] == 'bluestore': + notario.validate(host_vars, lvm_bluestore_scenario, defined_keys=True) except Invalid as error: display.vvv("Notario Failure: %s" % str(error))