Skip to content

Commit

Permalink
Fix permission for /var/log/kolla folder
Browse files Browse the repository at this point in the history
Docker will change the name volume folder permission to image already
set. /var/log/kolla is not created and well configured in image.

This patch set creates and configures proper permission /var/log/kolla
in base image.
This patch set also fixed the ceph image by adding ceph user to kolla
group.

TrivialFix

Change-Id: Ib5ef0187e90de2699b3cda31e819b4babb07e0af
  • Loading branch information
jeffrey4l authored and Jeffrey Zhang committed Sep 22, 2016
1 parent d10015d commit 73a39db
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 3 additions & 0 deletions docker/base/Dockerfile.j2
Expand Up @@ -282,6 +282,9 @@ RUN touch /usr/local/bin/kolla_extend_start \
&& chmod 755 /usr/local/bin/kolla_start /usr/local/bin/kolla_extend_start /usr/local/bin/kolla_set_configs \
&& chmod 440 /etc/sudoers \
&& groupadd kolla \
&& mkdir -p /var/log/kolla \
&& chown :kolla /var/log/kolla \
&& chmod 2775 /var/log/kolla \
&& rm -f /tmp/kolla_bashrc \
&& curl -sSL https://github.com/Yelp/dumb-init/releases/download/v1.1.3/dumb-init_1.1.3_amd64 -o /usr/local/bin/dumb-init \
&& chmod +x /usr/local/bin/dumb-init
Expand Down
3 changes: 2 additions & 1 deletion docker/ceph/ceph-base/Dockerfile.j2
Expand Up @@ -25,7 +25,8 @@ MAINTAINER {{ maintainer }}
{{ macros.install_packages(ceph_base_packages | customizable("packages")) }}

COPY extend_start.sh /usr/local/bin/kolla_extend_start
RUN chmod 755 /usr/local/bin/kolla_extend_start
RUN chmod 755 /usr/local/bin/kolla_extend_start \
&& usermod -a -G kolla ceph

{% block ceph_base_footer %}{% endblock %}
{% block footer %}{% endblock %}
Expand Down

0 comments on commit 73a39db

Please sign in to comment.