From 8d218253fdae2a53216842ecaacf47a4a645ad56 Mon Sep 17 00:00:00 2001
From: Matt Thompson <mattt@defunct.ca>
Date: Mon, 19 Oct 2015 14:40:15 +0100
Subject: [PATCH] Standardise ownership of *_venv_bin directories

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
---
 tasks/nova_pre_install.yml | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/tasks/nova_pre_install.yml b/tasks/nova_pre_install.yml
index 6d16e613..8f66a0da 100644
--- a/tasks/nova_pre_install.yml
+++ b/tasks/nova_pre_install.yml
@@ -75,10 +75,8 @@
   file:
     path: "{{ item.path }}"
     state: directory
-    owner: "{{ item.owner|default(nova_system_user_name) }}"
-    group: "{{ item.group|default(nova_system_group_name) }}"
   with_items:
-    - { path: "/openstack/venvs", mode: "0755", owner: "root", group: "root" }
+    - { path: "/openstack/venvs" }
     - { path: "{{ nova_venv_bin }}" }
   when: nova_venv_enabled | bool
   tags: