-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ROSE Server in a Docker Container #309
base: master
Are you sure you want to change the base?
Conversation
Fixes RedHat-Israel#173 Uses official CentOS Python 2.7 image.
ENABLE_PIPENV=true | ||
|
||
# Install pipenv | ||
RUN pip install --upgrade pipenv |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When we used a container, we don't need pipenv.
|
||
# Required env vars | ||
ENV LD_LIBRARY_PATH="/opt/rh/python27/root/usr/lib64/" \ | ||
ENABLE_PIPENV=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need LD_LIBRARY_PATH?
COPY . ./ | ||
|
||
# Install dependencies from pipfile | ||
RUN pipenv install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can install the requirements manually, or using a requirements.txt file generated
by pipenv.
|
||
# Server command | ||
ENTRYPOINT [ "pipenv" ] | ||
CMD [ "run", "python", "rose-server"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And without pipeenv, we can just run rose-server here.
Fixes #173
Uses official CentOS Python 2.7 image.
Verified building and running locally and can access ROSE Server on http://locahlost:8880