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.

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

Remplacer le container docker pour PostGIS en CI #1517

Closed
thbar opened this issue Feb 19, 2021 · 1 comment
Closed

Remplacer le container docker pour PostGIS en CI #1517

thbar opened this issue Feb 19, 2021 · 1 comment
Labels
dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité

Comments

@thbar
Copy link
Contributor

thbar commented Feb 19, 2021

Le setup CircleCI s'appuie sur un container Docker qui n'est plus mis à jour (https://hub.docker.com/r/mdillon/postgis/tags?page=1&ordering=last_updated) et avec Postgres 11, là où notre production est sur Postgres 12.

defaults_with_postgres: &defaults_with_postgres
<<: *defaults
docker:
- *default_docker
- image: mdillon/postgis:11-alpine
environment:
POSTGRES_USER: root
POSTGRES_DB: transport_test
defaults_with_postgres_and_phantom: &defaults_with_postgres_and_phantom
<<: *defaults
docker:
- *default_docker
- image: selenium/standalone-chrome:86.0
- image: mdillon/postgis:11-alpine
environment:
POSTGRES_USER: root
POSTGRES_DB: transport_test

Il faudra adapter la build et utiliser plutôt les containers officiels:

https://registry.hub.docker.com/r/postgis/postgis/tags?page=1&ordering=last_updated

@thbar thbar added the dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité label Feb 19, 2021
@thbar
Copy link
Contributor Author

thbar commented Apr 8, 2021

J'ai vérifié ce que nous utilisons en production aujourd'hui:

  • Postgres 12.4
  • PostGIS (avec SELECT PostGIS_Version();) 3.0 USE_GEOS=1 USE_PROJ=1 USE_STATS=1

thbar added a commit that referenced this issue Apr 8, 2021
The previous one is not updated.
@thbar thbar closed this as completed in a6939f7 Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité
Projects
None yet
Development

No branches or pull requests

1 participant