From 828714fafa60a83267091cf553564396a8bd2e58 Mon Sep 17 00:00:00 2001
From: Fatih Degirmenci <fdegir@gmail.com>
Date: Thu, 14 May 2020 17:14:26 +0000
Subject: [PATCH] Clone repos to temporary folder during packaging

Change-Id: I1fbdf892130b0d91de1d13e8261abb7b46d9213b
---
 playbooks/bootstrap-swconfig.yaml             |  2 ++
 playbooks/package.yaml                        |  2 ++
 .../package/tasks/prepare-packaging.yaml      | 28 ++++++++++---------
 3 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/playbooks/bootstrap-swconfig.yaml b/playbooks/bootstrap-swconfig.yaml
index 5b42dba..bd8d24f 100644
--- a/playbooks/bootstrap-swconfig.yaml
+++ b/playbooks/bootstrap-swconfig.yaml
@@ -24,3 +24,5 @@
 
   roles:
     - role: bootstrap-swconfig
+
+# vim: set ts=2 sw=2 expandtab:
diff --git a/playbooks/package.yaml b/playbooks/package.yaml
index 2d5f5b4..1cf4c34 100644
--- a/playbooks/package.yaml
+++ b/playbooks/package.yaml
@@ -24,3 +24,5 @@
 
   roles:
     - role: package
+
+# vim: set ts=2 sw=2 expandtab:
diff --git a/playbooks/roles/package/tasks/prepare-packaging.yaml b/playbooks/roles/package/tasks/prepare-packaging.yaml
index 48428e3..48ceebc 100644
--- a/playbooks/roles/package/tasks/prepare-packaging.yaml
+++ b/playbooks/roles/package/tasks/prepare-packaging.yaml
@@ -17,20 +17,16 @@
 # SPDX-License-Identifier: Apache-2.0
 # ============LICENSE_END=========================================================
 
-- name: Ensure provisioner and installer folders exist and empty
-  file:
-    path: "{{ engine_path }}/engine/{{ item.path}}"
-    state: "{{ item.state }}"
-  with_items:
-    - {path: "{{ engine_path }}/engine/provisioner", state: absent}
-    - {path: "{{ engine_path }}/engine/provisioner", state: directory}
-    - {path: "{{ engine_path }}/engine/installer", state: absent}
-    - {path: "{{ engine_path }}/engine/installer", state: directory}
+- name: Create temporary folder to clone engine repositories
+  tempfile:
+    prefix: "engine."
+    state: directory
+  register: tempdir
 
 - name: Clone engine provisioner repositories
   git:
     repo: "{{ provisioners[provisioner.key].src }}"
-    dest: "{{ engine_path }}/engine/provisioner/{{ provisioner.key }}"
+    dest: "{{ tempdir.path }}/{{ provisioner.key }}"
     version: "{{ provisioners[provisioner.key].version }}"
     refspec: "{{ provisioners[provisioner.key].refspec | default(omit) }}"
     force: true
@@ -44,7 +40,7 @@
 
 - name: Copy engine provisioner vars file into group_vars
   copy:
-    src: "{{ engine_path }}/engine/provisioner/{{ provisioner.key }}/vars/{{ provisioner.key }}.yaml"
+    src: "{{ tempdir.path }}/{{ provisioner.key }}/vars/{{ provisioner.key }}.yaml"
     dest: "{{ engine_path }}/engine/inventory/group_vars/all/{{ provisioner.key }}.yaml"
     force: true
   with_dict: "{{ provisioners }}"
@@ -54,7 +50,7 @@
 - name: Clone engine installer repositories
   git:
     repo: "{{ installers[installer.key].src }}"
-    dest: "{{ engine_path }}/engine/installer/{{ installer.key }}"
+    dest: "{{ tempdir.path }}/{{ installer.key }}"
     version: "{{ installers[installer.key].version }}"
     refspec: "{{ installers[installer.key].refspec | default(omit) }}"
     force: true
@@ -68,13 +64,19 @@
 
 - name: Copy engine installer vars file into group_vars
   copy:
-    src: "{{ engine_path }}/engine/installer/{{ installer.key }}/vars/{{ installer.key }}.yaml"
+    src: "{{ tempdir.path }}/{{ installer.key }}/vars/{{ installer.key }}.yaml"
     dest: "{{ engine_path }}/engine/inventory/group_vars/all/{{ installer.key }}.yaml"
     force: true
   with_dict: "{{ installers }}"
   loop_control:
     loop_var: installer
 
+- name: Delete temporary folder
+  file:
+    path: "{{ tempdir.path }}"
+    state: absent
+    force: true
+
 - name: Include vars from collected provisioners and installers vars files
   include_vars:
     dir: "{{ engine_path }}/engine/inventory/group_vars/all"
-- 
2.25.1