Skip to content

Commit

Permalink
Updated s2i Dockerfiles to build using requirements.txt (#1160)
Browse files Browse the repository at this point in the history
* Updated GPU wrapper to uninstall TF without GPU

* Updated GPU wrapper

* Updated Python-GPU Wrapper

* Integrated Python GPU wrapper into main Python folder"

* Removed old Python-GPU folder

* Setting TF-GPU to version 1.13.1

* Merged with master

* Python dependencies added as requirements.txt files

* Updated to master

* updated s2i Python scripts to include requirements.txt

* Pinned seldon core versions to 0.5.1
  • Loading branch information
JoelH96 authored and axsaucedo committed Dec 2, 2019
1 parent f9972d3 commit 13c734d
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 7 deletions.
3 changes: 2 additions & 1 deletion wrappers/s2i/python-conda/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ WORKDIR /microservice

COPY ./s2i/bin/ /s2i/bin

RUN pip install seldon-core
COPY requirements.txt ./requirements.txt
RUN pip install -r requirements.txt

EXPOSE 5000
1 change: 1 addition & 0 deletions wrappers/s2i/python-conda/requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
seldon-core==0.5.1
7 changes: 2 additions & 5 deletions wrappers/s2i/python/Dockerfile.gpu.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,8 @@ WORKDIR /microservice
COPY ./s2i/bin/ /s2i/bin

# keep install of seldon-core after the COPY to force re-build of layer
RUN pip3 install seldon-core
RUN pip3 install tensorflow-gpu==1.13.1

# Numpy version 1.16.1 required for tf-gpu
RUN pip3 install numpy==1.16.1
COPY requirements_gpu.txt ./requirements.txt
RUN pip3 install -r requirements.txt


EXPOSE 5000
3 changes: 2 additions & 1 deletion wrappers/s2i/python/Dockerfile.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ WORKDIR /microservice
COPY ./s2i/bin/ /s2i/bin

# keep install of seldon-core after the COPY to force re-build of layer
RUN pip install seldon-core
COPY requirements.txt ./requirements.txt
RUN pip install -r requirements.txt

EXPOSE 5000
1 change: 1 addition & 0 deletions wrappers/s2i/python/requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
seldon-core==0.5.1
4 changes: 4 additions & 0 deletions wrappers/s2i/python/requirements_gpu.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
seldon-core==0.5.1
tensorflow-gpu==1.13.1
numpy==1.16.1

0 comments on commit 13c734d

Please sign in to comment.