From 6ca0fcff0cf9e29f588946e8c3199744f543d6ec Mon Sep 17 00:00:00 2001
From: Kostiantyn Kalynovskyi <kkalinovskiy@gmail.com>
Date: Fri, 18 Jun 2021 04:03:39 +0000
Subject: [PATCH] Remove vino pool mounts

Change-Id: If02d95c925dc1e8b537e1609fea314b5b60ed36f
---
 config/manager/daemonset-template.yaml                   | 8 --------
 .../playbooks/roles/libvirt/tasks/create-storage.yaml    | 9 ++++++++-
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/config/manager/daemonset-template.yaml b/config/manager/daemonset-template.yaml
index 7c7b5ef..77f35d8 100644
--- a/config/manager/daemonset-template.yaml
+++ b/config/manager/daemonset-template.yaml
@@ -51,8 +51,6 @@ spec:
               mountPath: /sys/fs/cgroup
             - name: logs
               mountPath: /var/log/libvirt
-            - name: var-lib-vino-pool
-              mountPath: /var/lib/libvirt/vino-pool
             - name: etc-qemu
               mountPath: /etc/libvirt/qemu
             - name: etc-nwfilter
@@ -151,8 +149,6 @@ spec:
               mountPath: /var/lib/libvirt
             - name: var-run-libvirt
               mountPath: /var/run/libvirt
-            - name: var-lib-vino-pool
-              mountPath: /var/lib/libvirt/vino-pool
             - name: run
               mountPath: /run
             - name: dev
@@ -201,10 +197,6 @@ spec:
             defaultMode: 0555
         - name: pod-tmp
           emptyDir: {}
-        - name: var-lib-vino-pool
-          hostPath:
-            path: /var/lib/vino-pool
-            type: DirectoryOrCreate
         - name: etc-qemu
           hostPath:
             path: /etc/vino-qemu
diff --git a/vino-builder/assets/playbooks/roles/libvirt/tasks/create-storage.yaml b/vino-builder/assets/playbooks/roles/libvirt/tasks/create-storage.yaml
index 64dfd23..4f73b68 100644
--- a/vino-builder/assets/playbooks/roles/libvirt/tasks/create-storage.yaml
+++ b/vino-builder/assets/playbooks/roles/libvirt/tasks/create-storage.yaml
@@ -4,12 +4,19 @@
     command: facts
     uri: "{{ libvirt_uri }}"
 
+- name: create directory for vino pool
+  file:
+    state: directory
+    # TODO remove this hardcode, define vino storage variables in vino CR
+    path: "/var/lib/libvirt/vino-pool"
+    mode: 0755
+
 - name: define storage the storage pool
   virt_pool:
     state: present
     name: "{{ item.name }}"
     uri: "{{ libvirt_uri }}"
-    xml: "{{item.libvirtTemplate}}"
+    xml: "{{ item.libvirtTemplate }}"
 
 - name: activate the storage pool
   virt_pool: