Merge pull request #37 from laddp/boot_cdrom

Boot cdrom

Author: markgoddard

README.md

@@ -79,8 +79,8 @@ Role Variables
           allocated.
         - `name`: Name to associate with the volume being created; For `file` type volumes include extension if you would like volumes created with one.
         - `file_path`: Where the image of `file` type volumes should be placed; defaults to `libvirt_volume_default_images_path`
-        - `capacity`: volume capacity (can be suffixed with M,G,T or MB,GB,TB, etc)
-        - `device`: `disk`
+        - `device`: `disk` or `cdrom`
+        - `capacity`: volume capacity (can be suffixed with M,G,T or MB,GB,TB, etc) (required when type is `disk`)
         - `format`: options include `raw`, `qcow2`, `vmdk`.  See `man virsh` for the
           full range.  Default is `qcow2`
         - `image`: (optional) a URL to an image with which the volume is initalised (full copy).

tasks/volumes.yml

@@ -26,6 +26,7 @@
     -b {{ item.backing_image }}
     {% endif %}
   with_items: "{{ volumes }}"
+  when: "{{ item.device | default(libvirt_volume_default_device) == 'disk' }}"
   environment: "{{ libvirt_vm_script_env }}"
   register: volume_result
   changed_when:

templates/vm.xml.j2

@@ -14,6 +14,7 @@
     <type arch='{{ libvirt_vm_arch }}'{% if machine is not none %} machine='{{ machine }}'{% endif %}>hvm</type>
     <bootmenu enable='no'/>
     <boot dev='hd'/>
+    <boot dev='cdrom'/>
     <boot dev='network'/>
     <bios useserial='yes'/>
   </os>