Skip to content

Commit

Permalink
cmd/storagenode: move supervisord config file to storagenode-base image
Browse files Browse the repository at this point in the history
The supervisord.conf file is edited to set the args for the storagenode and storagenode-updater binaries at runtime. This change moves the config file to the base image so we can set the permission to allow non-root users edit the config file.

Non-root user permission is also needed for the /app directory so we can install/update the binaries when run as a non-root user.

Updates #4489

Change-Id: If7a51a00ea171253e41923501174a43393f4638c
  • Loading branch information
profclems authored and ihaid committed Mar 16, 2022
1 parent bf8ed28 commit 5a3047c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
4 changes: 3 additions & 1 deletion cmd/storagenode/Dockerfile.base
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,6 @@ ARG DOCKER_ARCH

FROM ${DOCKER_ARCH:-amd64}/alpine:3.15
RUN apk --no-cache -U add ca-certificates supervisor
RUN mkdir -p /var/log/supervisor
RUN mkdir -p /var/log/supervisor /app
COPY cmd/storagenode/docker-base/ /
RUN chmod a+rw /etc/supervisor/supervisord.conf /app
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
[supervisord]
user=root
nodaemon=true
logfile=/var/log/supervisor/supervisord.log
pidfile=/run/supervisord.pid
Expand Down

0 comments on commit 5a3047c

Please sign in to comment.