Merge pull request #217 from richm/bz1985572-1985571

omit unnecessary conditional - deadcode reported by static scanner

Author: richm

library/blivet.py

@@ -724,13 +724,13 @@ def _create(self):
         try:
             if create_vdo:
                 device = self._blivet.new_lv(name=self._volume['name'], vdo_lv=create_vdo,
-                                             parents=[vdopool if create_vdo else parent],
+                                             parents=[vdopool],
                                              size=size, fmt=fmt)
             else:
                 # This is here for backwards compatibility. Until 8.4 blivet does not support
                 # vdo_lv optional parameter
                 device = self._blivet.new_lv(name=self._volume['name'],
-                                             parents=[vdopool if create_vdo else parent],
+                                             parents=[parent],
                                              size=size, fmt=fmt)
         except Exception as e:
             raise BlivetAnsibleError("failed to set up volume '%s': %s" % (self._volume['name'], str(e)))

tests/tests_lvm_percent_size_nvme_generated.yml

@@ -0,0 +1,9 @@
+---
+# This file was generated by generate_tests.py
+- hosts: all
+  tasks:
+    - name: set disk interface for test
+      set_fact:
+        storage_test_use_interface: "nvme"
+
+- import_playbook: tests_lvm_percent_size.yml

tests/tests_lvm_percent_size_scsi_generated.yml

@@ -0,0 +1,9 @@
+---
+# This file was generated by generate_tests.py
+- hosts: all
+  tasks:
+    - name: set disk interface for test
+      set_fact:
+        storage_test_use_interface: "scsi"
+
+- import_playbook: tests_lvm_percent_size.yml