-
Notifications
You must be signed in to change notification settings - Fork 209
docs: Updated the Docker setup for portal #610
base: develop
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1,30 @@ | ||
FROM python:2.7 | ||
# getting the base Image | ||
FROM python:3.6-slim-buster | ||
|
||
MAINTAINER Ana Balica <[email protected]> | ||
|
||
EXPOSE 8000 | ||
|
||
# Initializing the working directory | ||
WORKDIR /usr/src | ||
|
||
RUN mkdir portal | ||
|
||
RUN cd portal | ||
|
||
# Re-Initializing the working directory | ||
WORKDIR /usr/src/portal | ||
|
||
# Copy the requirement.txt for installing | ||
COPY requirements/prod.txt /usr/src/portal/requirements/prod.txt | ||
RUN pip install -r requirements/prod.txt | ||
COPY requirements/dev.txt /usr/src/portal/requirements/dev.txt | ||
|
||
# Installing the requirements | ||
RUN pip install -r requirements/dev.txt | ||
|
||
# Installing the GDAL Library | ||
# RUN apt-get install python-gdal | ||
RUN apt-get update | ||
RUN apt-get install -y software-properties-common | ||
RUN add-apt-repository ppa:ubuntugis/ppa | ||
RUN apt-get install -y gdal-bin python-gdal python3-gdal | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes because while building it gives me an error regarding gdal lib not found during executing the cities_light command. So I tried to include and it resolved error. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Got you! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
are you sure about "python-gdal" as image using here is python3.6 , then why we need "python-gdal" library of python 2 . Also , installing "python-gdal" here will also install python2 and other dependency in the image . So, My suggestion is to only use "python3-gdal" here and verify for issue and also sure image size will further reduce. |
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.
It would be better we have an official Email ID here
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.
Yes so I can remove my email.