Merge pull request #7 from w-miller/optional-power-on

Make 'start' and 'autostart' configurable

Author: markgoddard

README.md

@@ -86,6 +86,12 @@ Role Variables
     - `console_log_path`: Path to console log file. Default is
       `{{ libvirt_vm_default_console_log_dir }}/{{ name }}-console.log`.
 
+    - `start`: Whether to immediately start the VM after defining it. Default
+      is `true`.
+
+    - `autostart`: Whether to start the VM when the host starts up. Default is
+      `true`.
+
 
 N.B. the following variables are deprecated: `libvirt_vm_state`,
 `libvirt_vm_name`, `libvirt_vm_memory_mb`, `libvirt_vm_vcpus`,

tasks/main.yml

@@ -24,6 +24,8 @@
     cpu_mode: "{{ vm.cpu_mode | default(cpu_mode_default, true) }}"
     volumes: "{{ vm.volumes | default([], true) }}"
     interfaces: "{{ vm.interfaces | default([], true) }}"
+    start: "{{ vm.start | default(true) }}"
+    autostart: "{{ vm.autostart | default(true) }}"
   with_items: "{{ libvirt_vms }}"
   loop_control:
     loop_var: vm

tasks/vm.yml

@@ -14,7 +14,7 @@
   include_tasks: check-interface.yml
   vars:
     interface: "{{ item }}"
-  with_items: "{{ vm.interfaces }}"
+  with_items: "{{ interfaces }}"
 
 - name: Ensure the VM is defined
   virt:
@@ -26,6 +26,6 @@
 - name: Ensure the VM is running and started at boot
   virt:
     name: "{{ vm.name }}"
-    autostart: true
-    state: running
+    autostart: "{{ autostart | bool }}"
+    state: "{{ 'running' if (start | bool) else 'shutdown' }}"
   become: true