fix: add condition to execute socket services

When listen_address is not defined (default), the drop-in files are not
created. Make sure the role will not try to manage the non-existing
services.

Author: btravouillon

handlers/main.yml

@@ -20,6 +20,9 @@
     state: "{{ item.enabled | bool | ternary('started', 'stopped') }}"
   become: true
   loop: "{{ _libvirt_socket_services }}"
+  when:
+    - item.listen_address is not none
+    - item.listen_address | length > 0
   loop_control:
     label: "{{ item.service }}"
   listen:

tasks/config.yml

@@ -139,15 +139,13 @@
     name: "{{ item.service }}"
     state: "{{ item.enabled | bool | ternary('started', 'stopped') }}"
     enabled: "{{ item.enabled | bool }}"
-  become: True
-  loop: "{{ _libvirt_services }}"
+  become: true
+  loop: "{{ _libvirt_services + _libvirt_socket_services }}"
+  when: item.listen_address is not defined or
+        (item.listen_address is not none and item.listen_address | length > 0)
   loop_control:
     label: "{{ item.service }}"
   vars:
     _libvirt_services:
-      - service: libvirtd-tcp.socket
-        enabled: "{{ libvirt_host_tcp_listen | bool }}"
-      - service: libvirtd-tls.socket
-        enabled: "{{ libvirt_host_tls_listen | bool }}"
       - service: libvirtd
         enabled: true