Fix sudo permissions for elements and scripts dir
The reason we didn't catch this is because the jenkins slaves already have these folders created. In a future patch we should move these into /var/lib/nodepool Change-Id: I01349f6bc21002e638ecbd301f4fb6269df37376 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
parent
c444df1bf9
commit
ef178a3788
@ -63,7 +63,7 @@
|
||||
src: "{{ nodepool_file_logging_conf }}"
|
||||
|
||||
- name: Create nodepool elements directory.
|
||||
become: no
|
||||
become: yes
|
||||
file:
|
||||
group: "{{ ansible_user }}"
|
||||
mode: 0775
|
||||
@ -84,7 +84,7 @@
|
||||
when: nodepool_elements_dir != ""
|
||||
|
||||
- name: Create nodepool scripts directory.
|
||||
become: no
|
||||
become: yes
|
||||
file:
|
||||
group: "{{ ansible_user }}"
|
||||
mode: 0775
|
||||
|
7
tox.ini
7
tox.ini
@ -9,8 +9,11 @@ deps = -r{toxinidir}/requirements.txt
|
||||
|
||||
[testenv:functional]
|
||||
commands =
|
||||
ansible-playbook -i tests/inventory tests/test.yaml
|
||||
passenv = HOME
|
||||
# NOTE(pabelanger): Because ansible default ansible_user to null now, we need to pass it via CLI.
|
||||
ansible-playbook -i tests/inventory tests/test.yaml -e ansible_user={env:USER}
|
||||
passenv =
|
||||
HOME
|
||||
USER
|
||||
setenv =
|
||||
ANSIBLE_CONFIG = {toxinidir}/tests/ansible.cfg
|
||||
PYTHONUNBUFFERED = 1
|
||||
|
Loading…
x
Reference in New Issue
Block a user