-
Notifications
You must be signed in to change notification settings - Fork 103
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.
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
Feat/docker2 #2623
Feat/docker2 #2623
Conversation
Codecov ReportPatch has no changes to coverable lines. 📢 Thoughts on this report? Let us know!. |
c2c31ec
to
6409f15
Compare
5afb9ad
to
47b0ebc
Compare
backend/Dockerfile
Outdated
RUN apt-get update && apt-get upgrade -y && apt-get -y install wget lsb-release gnupg | ||
RUN echo "deb http://apt.postgresql.org/pub/repos/apt $(lsb_release -cs)-pgdg main" > /etc/apt/sources.list.d/pgdg.list && \ | ||
wget --quiet -O - https://www.postgresql.org/media/keys/ACCC4CF8.asc | apt-key add - | ||
RUN apt-get update && apt-get install -y postgresql-client postgis |
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.
On avait dit qu'on n'installait pas PostgreSQL et PostGIS dans l'image de GeoNature, pour ne pas l'alourdir et la compliquer, non ? #2615
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.
on pourrait installer seulement postgres-client qui est léger. Et d'autre part proposer directement fichier sql plutot qu'on fichier .asc depuis le serveur geonature http://geonature.fr/data/ign/
cela permettrait de se passer de postgis (pour la commande raster2pgsql)
514781d
to
083d25b
Compare
7702345
to
b665972
Compare
d79715a
to
39bc758
Compare
4b7469b
to
a91299a
Compare
a91299a
to
8cf7034
Compare
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.
Penser à virer le dernier commit « WIP push images for testing purpose » avant de merger
8cf7034
to
3d4325a
Compare
No description provided.