From: Andrew Schoen <aschoen@redhat.com>
Date: Fri, 4 Aug 2017 16:14:29 +0000 (-0500)
Subject: ceph-common: render ceph_conf_overrides into fetch_directory
X-Git-Tag: v3.0.0rc1~2^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F1745%2Fhead;p=ceph-ansible.git

ceph-common: render ceph_conf_overrides into fetch_directory

Writing into /tmp is not always allowed, but we can assume the
fetch_directory is writable.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
---

diff --git a/roles/ceph-common/tasks/generate_ceph_conf.yml b/roles/ceph-common/tasks/generate_ceph_conf.yml
index f23a485d9..d0803f794 100644
--- a/roles/ceph-common/tasks/generate_ceph_conf.yml
+++ b/roles/ceph-common/tasks/generate_ceph_conf.yml
@@ -11,11 +11,13 @@
     - /etc/ceph/ceph.d/
 
 - name: template ceph_conf_overrides
-  local_action: copy content="{{ ceph_conf_overrides }}" dest=/tmp/ceph_conf_overrides_temp
+  local_action: copy content="{{ ceph_conf_overrides }}" dest="{{ fetch_directory }}/ceph_conf_overrides_temp"
+  become: false
   run_once: true
 
 - name: get rendered ceph_conf_overrides
-  local_action: set_fact ceph_conf_overrides_rendered="{{ lookup('template', '/tmp/ceph_conf_overrides_temp') | from_yaml }}"
+  local_action: set_fact ceph_conf_overrides_rendered="{{ lookup('template', '{{ fetch_directory }}/ceph_conf_overrides_temp') | from_yaml }}"
+  become: false
   run_once: true
 
 - name: "generate ceph configuration file: {{ cluster }}.conf"