From d16156584aa653ed5ddc92fc59dccab92b94e718 Mon Sep 17 00:00:00 2001
From: Julia Kreger <juliaashleykreger@gmail.com>
Date: Wed, 29 Nov 2017 09:55:42 -0500
Subject: [PATCH] Fix logging permissions for IPA

Since the change to Zuulv3 based CI, we've observed some
changes in default permissions which cause some CI jobs to
now fail.

As such, adding the creation of /var/log/ironic the setup of
ironic.

Change-Id: I8b6dbb724c6de618798d4207fbfacc935837ae57
---
 .../bifrost-ironic-install/tasks/bootstrap.yml     | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml b/playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml
index 4bad7f1e8..534383398 100644
--- a/playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml
+++ b/playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml
@@ -182,13 +182,16 @@
     state: directory
     mode: 0755
   when: init_template == 'systemd_template.j2'
+
 - name: "Install ironic-inspector to permit use of inspection interface"
   include: inspector_bootstrap.yml
   when: enable_inspector | bool == true
+
 - name: "Get ironic-api & ironic-conductor install location"
   shell: echo $(dirname $(which ironic-api))
   register: ironic_install_prefix
   environment: "{{ bifrost_venv_env if enable_venv else {} }}"
+
 - name: "Set permissions for /var/lib/ironic for the ironic user"
   file:
     path: "{{ item }}"
@@ -199,6 +202,17 @@
   with_items:
     - "/var/lib/ironic"
     - "/var/lib/ironic/images"
+
+- name: "Set permissions for /var/log/ironic for the ironic user"
+  file:
+    path: "{{ item }}"
+    state: directory
+    mode: 0755
+    owner: "ironic"
+    group: "ironic"
+  with_items:
+    - "/var/log/ironic"
+
 - name: "Place ironic services"
   template:
     src: "{{ init_template }}"