Skip to content
This repository has been archived by the owner on Jul 29, 2020. It is now read-only.

Modified the user as cf instead of root #175

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion build-tools/Dockerfile.runtime
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@ FROM python:2.7-alpine3.7
ENV APPPATH /app

RUN mkdir -p "$APPPATH/bin" \
&& chmod -R 755 "$APPPATH"
&& chmod -R 755 "$APPPATH" \
&& adduser -D cf
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's change the user name to ctlr

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure I will change it


RUN pip install "pip==18.0.0"

WORKDIR $APPPATH

Expand All @@ -16,6 +19,7 @@ RUN apk --no-cache --update add --virtual pip-install-deps git && \
pip install --process-dependency-links -r /tmp/requirements.txt && \
apk del pip-install-deps

USER cf
# Run the run application in the projects bin directory.
EXPOSE 8080
CMD [ "/app/bin/cf-bigip-ctlr" ]