Pass yamlint on the role

Correct these warning/errors:
- truthy value should be one of [false, true]  (truthy)
- missing starting space in comment  (comments)
- comment not indented like content  (comments-indentation)
- trailing spaces  (trailing-spaces)

Author: Christian Iuga

defaults/main.yml

@@ -6,7 +6,7 @@ libvirt_vm_default_console_log_dir: "/var/log/libvirt-consoles"
 
 # Whether UUID should be calculated by hashing the VM name. If not, the UUID is
 # randomly generated by libvirt when the VM is defined.
-libvirt_vm_default_uuid_deterministic: False
+libvirt_vm_default_uuid_deterministic: false
 
 # The default location for libvirt images
 libvirt_volume_default_images_path: '/var/lib/libvirt/images'
@@ -42,7 +42,7 @@ libvirt_vm_emulator:
 # Default value for clock syncing. The default (false) uses <clock sync="localtime">
 # to configure the instances clock synchronisation. Change to a timezone to make
 # configuration use <clock offset="specified offset">
-libvirt_vm_clock_offset: False
+libvirt_vm_clock_offset: false
 
 # Default value for whether to trust guest receive filters. This gets mapped to
 # the trustGuestRxFilters attribute of VM interfaces.
@@ -53,7 +53,7 @@ libvirt_vm_trust_guest_rx_filters: false
 # the values of the deprecated variables below.
 # See README.md or tasks/main.yml for these attributes' defaults.
 libvirt_vms:
-      # State of the VM. May be 'present' or 'absent'.
+    # State of the VM. May be 'present' or 'absent'.
     - state: "{{ libvirt_vm_state }}"
 
       # Name of the VM.
@@ -85,7 +85,7 @@ libvirt_vms:
 
       # XML template file to source domain definition
       xml_file: vm.xml.j2
-      
+
       # May be one of: bios, or efi.
       boot_firmware: bios
 
@@ -113,6 +113,6 @@ libvirt_vm_volumes:
 libvirt_vm_interfaces:
 libvirt_vm_console_log_path:
 # Required, so we should fail if not set.
-#libvirt_vm_name:
-#libvirt_vm_memory_mb:
-#libvirt_vm_vcpus:
+# libvirt_vm_name:
+# libvirt_vm_memory_mb:
+# libvirt_vm_vcpus:

meta/main.yml

@@ -1,6 +1,6 @@
 ---
 galaxy_info:
-  #role_name: libvirt_vm
+  # role_name: libvirt_vm
   author: Mark Goddard
   description: >
     Role to configure and create VMs on a Libvirt/KVM hypervisor

tasks/check-usb-devices.yml

@@ -3,7 +3,7 @@
   command: lsusb  -d {{ usb_device.vendor }}:{{ usb_device.product }}
   register: host_attached_usb_device
   become: true
-  changed_when: false 
+  changed_when: false
   failed_when: false
 
 - name: Check USB device is present on Host system

tasks/destroy-vm.yml

@@ -7,15 +7,15 @@
     command: list_vms
     uri: "{{ libvirt_vm_uri | default(omit, true) }}"
   register: result
-  become: yes
+  become: true
 
 - block:
     - name: Ensure the VM is absent
       virt:
         name: "{{ vm.name }}"
         state: destroyed
         uri: "{{ libvirt_vm_uri | default(omit, true) }}"
-      become: yes
+      become: true
 
     # note(wszumski): the virt module does not seem to support
     # removing vms with nvram defined - as a workaround, use the
@@ -30,6 +30,6 @@
           undefine
           {% if boot_firmware == 'efi' %} --nvram{% endif %}
           {{ vm.name }}
-      become: yes
+      become: true
       changed_when: true
   when: vm.name in result.list_vms

tasks/destroy-volumes.yml

@@ -11,4 +11,4 @@
   changed_when:
     - volume_result is success
     - (volume_result.stdout | from_json).changed | default(True)
-  become: yes
+  become: true

tasks/volumes.yml

@@ -36,7 +36,7 @@
   changed_when:
     - volume_result is success
     - (volume_result.stdout | from_json).changed | default(True)
-  check_mode: False
+  check_mode: false
   become: true
 
 - name: Ensure the VM network volumes exist
@@ -49,5 +49,5 @@
   changed_when:
     - volume_result_network is success
     - volume_result_network.rc == 1
-  check_mode: False
+  check_mode: false
   become: true