fix: handle explicit False for bootable and revertable parameters

Fix boolean parameter checking to distinguish between explicit False and
unset (None) values. Previously, truthy checks treated False the same as
None, causing silent conflicts when:
- Global parameter set to False
- Set-level parameter set to True

The global False was silently ignored instead of raising a conflict error.

Changes:
- Use `module_args.get("param") is not None` instead of `if module_args["param"]:`
- Applies to both snapshot_lvm_bootable and snapshot_lvm_revertable
- Fixed in snapmgr.py (conflict detection) and validate.py (parameter propagation)

This ensures ERROR_BOOTABLE_CONFLICT and ERROR_REVERTABLE_CONFLICT are
correctly raised for all conflicting configurations, not just when the
global value is True.

Signed-off-by: Todd Gill <tgill@redhat.com>

Author: trgill

module_utils/snapshot_lsr/snapmgr.py

@@ -230,7 +230,7 @@ def mgr_snapshot_cmd(module, module_args, snapset_json):
     volume_list = snapset_json["volumes"]
 
     # Bootable global variable is set
-    if module_args["snapshot_lvm_bootable"]:
+    if module_args.get("snapshot_lvm_bootable") is not None:
         bootable = module_args["snapshot_lvm_bootable"]
 
     # Global is not set, check the snapset
@@ -252,7 +252,7 @@ def mgr_snapshot_cmd(module, module_args, snapset_json):
             }
 
     # Revertable global variable is set
-    if module_args["snapshot_lvm_revertable"]:
+    if module_args.get("snapshot_lvm_revertable") is not None:
         revertable = module_args["snapshot_lvm_revertable"]
 
     # Global is not set, check the snapset

module_utils/snapshot_lsr/validate.py

@@ -33,10 +33,10 @@ def get_json_from_args(module, module_args, vg_include):
     if module_args["snapshot_lvm_snapset_name"]:
         args_dict["name"] = module_args["snapshot_lvm_snapset_name"]
 
-    if module_args["snapshot_lvm_bootable"]:
+    if module_args.get("snapshot_lvm_bootable") is not None:
         args_dict["bootable"] = module_args["snapshot_lvm_bootable"]
 
-    if module_args["snapshot_lvm_revertable"]:
+    if module_args.get("snapshot_lvm_revertable") is not None:
         args_dict["revertable"] = module_args["snapshot_lvm_revertable"]
 
     for vg, lv_list in vgs_lvs_iterator(
@@ -341,10 +341,10 @@ def validate_snapset_json(cmd, module_args, verify_only):
 
     snapset_dict = module_args["snapshot_lvm_set"]
 
-    if module_args["snapshot_lvm_bootable"]:
+    if module_args.get("snapshot_lvm_bootable") is not None:
         snapset_dict["snapshot_lvm_bootable"] = module_args["snapshot_lvm_bootable"]
 
-    if module_args["snapshot_lvm_revertable"]:
+    if module_args.get("snapshot_lvm_revertable") is not None:
         snapset_dict["snapshot_lvm_revertable"] = module_args["snapshot_lvm_revertable"]
 
     if cmd == SnapshotCommand.SNAPSHOT: