Added blivet backwards compatibility for shared VGs

Older versions of blivet do not support 'shared' parameter. This resulted in
failures in tests unrelated to shared VGs. This change fixes that
behavior as well as fixes minor condition error in a test.

library/blivet.py

@@ -1534,7 +1534,14 @@ def _create(self):
         if not self._device:
             members = self._manage_encryption(self._create_members())
             try:
-                pool_device = self._blivet.new_vg(name=self._pool['name'], parents=members, shared=self._pool['shared'])
+                if self._pool['shared']:
+                    pool_device = self._blivet.new_vg(name=self._pool['name'],
+                                                      parents=members,
+                                                      shared=self._pool['shared'])
+                else:
+                    # Makes it compatible with older blivet versions that does not support shared VGs
+                    pool_device = self._blivet.new_vg(name=self._pool['name'],
+                                                      parents=members)
             except Exception as e:
                 raise BlivetAnsibleError("failed to set up pool '%s': %s" % (self._pool['name'], str(e)))
 

tests/test-verify-pool.yml

@@ -23,7 +23,7 @@
 
 - name: Verify that VG shared value checks out
   assert:
-    that: (storage_test_pool.shared | bool) and ('1' in vgs_dump.stdout)
+    that: (storage_test_pool.shared | bool) == ('1' in vgs_dump.stdout)
     msg: >-
       Shared VG presence ({{ storage_test_pool.shared }})
       does not match its expected state ({{ '1' in vgs_dump.stdout }})