openstack/tripleo-heat-templates
Code Issues Proposed changes

Fix idempotency for horizon container logs

Browse Source 
Whenever /var/log/horizon/horizon.log recreated as a root,
the touch command fails, and the chown commands got skipped.
This results in idempotency issues when the logs path become
accessed under mismatching user.

Fix this by making the chown command always applying disregard
of the touch command results.

Closes-Bug: #1819626

Change-Id: Ied538dbc7dbd311361108747a1e89d1f5740eedd
Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
This commit is contained in:
Bogdan Dobrelya 2019-03-12 11:38:51 +01:00
parent c59bf8ce13
commit 95e142d84e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
deployment/horizon/horizon-container-puppet.yaml
View File

@ -254,7 +254,7 @@ outputs:
# FIXME Apache needs to read files in /etc/openstack-dashboard
# Need to set permissions to match the BM case,
# http://paste.openstack.org/show/609819/
command: ['/bin/bash', '-c', 'touch /var/log/horizon/horizon.log && chown -R apache:apache /var/log/horizon && chmod -R a+rx /etc/openstack-dashboard']
command: ['/bin/bash', '-c', 'touch /var/log/horizon/horizon.log ; chown -R apache:apache /var/log/horizon && chmod -R a+rx /etc/openstack-dashboard']
volumes:
- /var/log/containers/horizon:/var/log/horizon:z
- /var/log/containers/httpd/horizon:/var/log/httpd:z