openstack/openstack-ansible-os_neutron
Code Issues Proposed changes

Standardise ownership of *_venv_bin directories

Browse Source 
This review standardises the ownership of the *_venv_dir directories by
ensuring all are created as root instead of using the service's user.
This behaviour already exists in the following roles:

- os_aodh
- os_glance
- os_swift

Change-Id: Idab42b83bece3624271780105f6a937eb88c7491
This commit is contained in:
Matt Thompson 2015-10-19 14:40:15 +01:00
parent c5dcce515f
commit 38e97b9fae
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tasks/neutron_pre_install.yml
View File

@ -58,10 +58,8 @@
file: file:
path: "{{ item.path }}" path: "{{ item.path }}"
state: directory state: directory
owner: "{{ item.owner|default(neutron_system_user_name) }}"
group: "{{ item.group|default(neutron_system_group_name) }}"
with_items: with_items:
- { path: "/openstack/venvs", mode: "0755", owner: "root", group: "root" } - { path: "/openstack/venvs" }
- { path: "{{ neutron_venv_bin }}" } - { path: "{{ neutron_venv_bin }}" }
when: neutron_venv_enabled | bool when: neutron_venv_enabled | bool
tags: tags: