ceph
/
ceph-ansible
mirror of https://github.com/ceph/ceph-ansible.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

common: do not become root on local task 

There is no need to become root on local_action. This will event trigger
an error on some systems as it will try to run a sudo command. If the
current user does not have passwordless sudo, Ansible will fail. Anyway
using the current user is perfectly fine and no elevation privilege is
needed.

Signed-off-by: Sébastien Han <seb@redhat.com>
pull/1198/head
Sébastien Han 2016-12-16 11:36:42 +01:00
parent faabfdcefe
commit 6c71cf5b10
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
roles/ceph-common/tasks/facts.yml
View File

@ -35,6 +35,7 @@
# We want this check to be run only on one mon # We want this check to be run only on one mon
- name: check if {{ fetch_directory }} directory exists - name: check if {{ fetch_directory }} directory exists
local_action: stat path="{{ fetch_directory }}/monitor_keyring.conf" local_action: stat path="{{ fetch_directory }}/monitor_keyring.conf"
become: false
register: monitor_keyring_conf register: monitor_keyring_conf
run_once: true run_once: true
@ -71,6 +72,7 @@
- name: write initial mon keyring in {{ fetch_directory }}/monitor_keyring.conf if it doesn't exist - name: write initial mon keyring in {{ fetch_directory }}/monitor_keyring.conf if it doesn't exist
local_action: shell echo {{ monitor_keyring.stdout }} | tee {{ fetch_directory }}/monitor_keyring.conf local_action: shell echo {{ monitor_keyring.stdout }} | tee {{ fetch_directory }}/monitor_keyring.conf
become: false
when: when:
- test_initial_monitor_keyring.rc == 0 - test_initial_monitor_keyring.rc == 0