ceph-validate : Added functions to accept true and flase

ceph-validate used to throw error for setting flags as 'true' or 'false' for True and False
Now user can set the flags 'dmcrypt' and 'osd_auto_discovery' as 'true' or 'false'

Will fix - Bug 1638325

Signed-off-by: VasishtaShastry <vipin.indiasmg@gmail.com>

diff --git a/plugins/actions/validate.py b/plugins/actions/validate.py
index e6e40f9fe671433db240d4f8e7dc4e1203004ee4..d48d581b12833893ac2e09a8d935d3c9311339c3 100644 (file)
--- a/plugins/actions/validate.py
+++ b/plugins/actions/validate.py
@@ -155,6 +155,14 @@ def validate_monitor_options(value):
     assert any([monitor_address_given, monitor_address_block_given, monitor_interface_given]), msg
 
 
+def validate_dmcrypt_bool_value(value):
+    assert value in ["true", True, "false", False], "dmcrypt can be set to true/True or false/False (default)"
+
+
+def validate_osd_auto_discovery_bool_value(value):
+    assert value in ["true", True, "false", False], "osd_auto_discovery can be set to true/True or false/False (default)"
+
+
 def validate_osd_scenarios(value):
     assert value in ["collocated", "non-collocated", "lvm"], "osd_scenario must be set to 'collocated', 'non-collocated' or 'lvm'"
 
@@ -222,8 +230,8 @@ rados_options = (
 )
 
 osd_options = (
-    (optional("dmcrypt"), types.boolean),
-    (optional("osd_auto_discovery"), types.boolean),
+    (optional("dmcrypt"), validate_dmcrypt_bool_value),
+    (optional("osd_auto_discovery"), validate_osd_auto_discovery_bool_value),
     ("osd_scenario", validate_osd_scenarios),
 )