Browse Source

Create ceph directories in config-download without become

Given how config-download runs in the main branch it's no longer
necessary to use become when creating the work directories for
ceph-ansible to be executed or when running the tripleo_ceph_client
role. Using become introduces the bug this change resolves. Also,
as we are not using become we won't set the owner of the directory.
Instead we will use the default owner of whoever created the directory.

Change-Id: I65cd66ed5c94b548b775b9b4829717c202837d7e
Closes-Bug: #1912103
changes/50/771150/3
John Fulton 3 months ago
parent
commit
ef33a0f91f
2 changed files with 0 additions and 4 deletions
  1. +0
    -2
      tripleo_ansible/roles/tripleo_ceph_client/tasks/fetch_dir.yml
  2. +0
    -2
      tripleo_ansible/roles/tripleo_ceph_work_dir/tasks/prepare.yml

+ 0
- 2
tripleo_ansible/roles/tripleo_ceph_client/tasks/fetch_dir.yml View File

@ -24,10 +24,8 @@
path: "{{ tripleo_ceph_client_fetch_dir }}/"
when: clean
- name: Create ceph_client_fetch_dir for the current execution
become: true
tags: 'build_fetch_dir'
file:
path: "{{ tripleo_ceph_client_fetch_dir }}"
state: directory
owner: "{{ ansible_user }}"
when: build

+ 0
- 2
tripleo_ansible/roles/tripleo_ceph_work_dir/tasks/prepare.yml View File

@ -15,11 +15,9 @@
# under the License.
- name: create ceph-ansible temp dirs
become: true
file:
path: "{{ item }}"
state: directory
owner: "{{ ansible_user }}"
with_items:
- "{{ playbook_dir }}/ceph-ansible"
- "{{ playbook_dir }}/ceph-ansible/group_vars"


Loading…
Cancel
Save