Skip to content

Commit 5211ff9

Browse files
committed
Make AppArmor configurable separately from daemon installation
This may be useful in cases where we don't want to install the daemon, but we do want to configure AppArmor for directory storage pools.
1 parent 8fbf801 commit 5211ff9

File tree

3 files changed

+7
-1
lines changed

3 files changed

+7
-1
lines changed

README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,9 @@ systemd.socket for format. Default is unset.
143143

144144
`libvirt_host_tls_cacert`: TLS CA certificate. Default is unset.
145145

146+
`libvirt_host_configure_apparmor`: Whether to configure AppArmor for directory
147+
storage pools.
148+
146149
Dependencies
147150
------------
148151

defaults/main.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,3 +105,6 @@ libvirt_host_tls_server_key:
105105
libvirt_host_tls_client_cert:
106106
libvirt_host_tls_client_key:
107107
libvirt_host_tls_cacert:
108+
109+
# Whether to configure AppArmor for directory storage pools.
110+
libvirt_host_configure_apparmor: "{{ libvirt_host_install_daemon | bool }}"

tasks/post-install-Debian.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
line: " {{ item.path }}/** rwk,"
3434
become: true
3535
when:
36-
- libvirt_host_install_daemon | bool
36+
- libvirt_host_configure_apparmor | bool
3737
- ansible_facts.apparmor.status | default == 'enabled'
3838
- item.type == "dir"
3939
loop: "{{ libvirt_host_pools | flatten(levels=1) }}"

0 commit comments

Comments
 (0)