Skip to content

froze pip package versions, upgraded pip, pointed to ubuntu 17 #4

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.

Sign up for GitHub

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

Open
DarthDippy wants to merge 1 commit into
base: master
Choose a base branch
from
Open

froze pip package versions, upgraded pip, pointed to ubuntu 17 #4

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
1 change: 1 addition & 0 deletions docker/conda/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ RUN /opt/conda/bin/conda install jupyter -y --quiet && mkdir /opt/notebooks

COPY ./source/python /python
WORKDIR /python
RUN pip install --upgrade pip
RUN pip install -r requirements.txt

CMD /opt/conda/bin/jupyter notebook --notebook-dir=/opt/notebooks --ip='*' --port=5002 --no-browser
Expand Down
3 changes: 2 additions & 1 deletion docker/flask/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
FROM ubuntu
FROM ubuntu:17.10
Copy link
Collaborator

@WanderingStar WanderingStar May 4, 2018

Choose a reason for hiding this comment

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

Why 17.10 and not 18.04? 18.04 is an LTS release, while 17.10 is not.

Copy link
Author

@DarthDippy DarthDippy May 4, 2018

Choose a reason for hiding this comment

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

18.04 was just released last week and I would want to fully test it. I didn't realize 17.10 was not lts.


RUN apt-get update -y && apt-get install -y python3 python3-pip python3-dev build-essential

COPY ./source/python /python
WORKDIR /python
RUN pip3 install --upgrade pip
RUN pip3 install -r requirements.txt

CMD python3 /python/d4d/flask_app.py
8 changes: 4 additions & 4 deletions source/python/requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Flask
records
psycopg2
pymongo
Flask==1.0.2
records==0.5.2
psycopg2-binary==2.7.4
pymongo==3.6.1