Skip to content
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

Updated s2i Dockerfiles to build using requirements.txt #1160

Merged
merged 14 commits into from
Dec 2, 2019
Merged

Updated s2i Dockerfiles to build using requirements.txt #1160

Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
f7bc182
Updated GPU wrapper to uninstall TF without GPU
JoelH96 Oct 29, 2019
1de6b15
Updated GPU wrapper
JoelH96 Nov 6, 2019
57e105b
Updated Python-GPU Wrapper
JoelH96 Nov 6, 2019
6c7da2c
Integrated Python GPU wrapper into main Python folder"
JoelH96 Nov 8, 2019
5d4cbad
Removed old Python-GPU folder
JoelH96 Nov 8, 2019
9910545
Merge remote-tracking branch 'upstream/master'
JoelH96 Nov 22, 2019
148e2b8
Setting TF-GPU to version 1.13.1
JoelH96 Nov 22, 2019
b2dd8b3
Merge remote-tracking branch 'upstream/master'
JoelH96 Nov 29, 2019
db20024
Merged with master
JoelH96 Nov 29, 2019
e7673eb
Python dependencies added as requirements.txt files
JoelH96 Nov 29, 2019
1a9cb1c
Merge pull request #1 from JoelH96/s2i-python-requirements
JoelH96 Nov 29, 2019
62c95d0
Updated to master
JoelH96 Nov 29, 2019
e39990f
updated s2i Python scripts to include requirements.txt
JoelH96 Nov 29, 2019
59ff11d
Pinned seldon core versions to 0.5.1
JoelH96 Nov 29, 2019
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
3 changes: 2 additions & 1 deletion wrappers/s2i/python-conda/Dockerfile
View file
Open in desktop
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
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
seldon-core
adriangonz marked this conversation as resolved.
Show resolved Hide resolved
7 changes: 2 additions & 5 deletions wrappers/s2i/python/Dockerfile.gpu.tmpl
View file
Open in desktop
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
View file
Open in desktop
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
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
seldon-core
4 changes: 4 additions & 0 deletions wrappers/s2i/python/requirements_gpu.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
seldon-core
tensorflow-gpu==1.13.1
numpy==1.16.1