Merge pull request #2 from Tcharl/logicalstorage

Logicalstorage
OsgiliathEnterprise · Apr 21, 2020 · 57ea475 · 57ea475
2 parents 9f47735 + 7013358
commit 57ea475
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -23,7 +23,7 @@ should be a dict containing the following items:
 - `owner` The owner of the pool. (only `dir`)
 - `group` The group of the pool. (only `dir`)
 - `source` The name of the volume group. (only `lvm2`)
-- `pvs` A list of physical volumes the volume group consists of. (only `lvm2`)
+- `pvs` A list of physical volumes the volume group consists of. (only `lvm2`). N.B. if specified, the lvg will be created on top of the PVS, otherwise the lv should have been created before.
 
 `libvirt_host_networks` is a list of networks to define and start. Each item
 should be a dict containing the following items:

diff --git a/tasks/pools.yml b/tasks/pools.yml
@@ -14,7 +14,7 @@
   lvg:
     vg: "{{ item.source }}"
     pvs: "{{ item.pvs }}"
-  when: item.type == "lvm2"
+  when: item.type == "lvm2" and item.pvs is defined
   with_items: "{{ libvirt_host_pools }}"
   become: True