config: Update Dockerfile to use Alpine image as a base #2
Conversation
… Python and Ruby Co-authored-by: Victoria-Mag <[email protected]> Co-authored-by: KuzzyKabana <[email protected]>
Dockerfile
Outdated
| RUN pip3 install --no-cache --upgrade pip setuptools | ||
|
|
||
| # Get speaker-verification repo | ||
| RUN git clone https://github.com/OnTrack-UG-Squad/speaker-verification.git |
There was a problem hiding this comment.
do you need to install any dependencies for this?
There was a problem hiding this comment.
Yes, I'll need to update it to make the virtual environment and install the requirements.txt stuff, as per the main speaker-verification readme instructions.
There was a problem hiding this comment.
Rather than going about the virtual environment stuff, we've just gone ahead and installed the dependencies. The docker container doesn't really need things to be kept in a virtual environment from what I can see.
We have since run into problems on the speaker-verification side of things. It appears some packages have since been updated and are now causing problems which we will have to investigate.
Co-authored-by: Victoria-Mag <[email protected]> Co-authored-by: KuzzyKabana <[email protected]>
kugamigh
force-pushed
the
config/update-docker-image
branch
from
0993820 to
d5e2f13
Compare
| RUN pip3 install --no-cache --upgrade pip setuptools | ||
|
|
||
| # Get speaker-verification repo | ||
| RUN git clone -b fix/logger-issues https://github.com/OnTrack-UG-Squad/speaker-verification.git |
There was a problem hiding this comment.
This is temporary until speaker-verification side of things is fixed.
Co-authored-by: Victoria-Mag <[email protected]> Co-authored-by: KuzzyKabana <[email protected]>
Co-authored-by: Victoria-Mag <[email protected]> Co-authored-by: KuzzyKabana <[email protected]>
Fixes #1
Proposed Changes
Co-authored-by: Victoria-Mag [email protected]
Co-authored-by: KuzzyKabana [email protected]