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

Réparer le docker compose #1783

Merged
merged 9 commits into from Sep 23, 2021
Merged

Réparer le docker compose #1783

merged 9 commits into from Sep 23, 2021

Conversation

fchabouis
Copy link
Contributor

Je me suis rendu compte qu'il n'était plus possible de lancer le site en local avec le docker compose.

  • le container dont on partait n'était plus le bon
  • certaines de nos dépendances elixir sont tirées depuis git, il faut donc installer git sur la machine
  • on utilise un script qui s'appelle wait-for-it mais qui n'était plus compatible avec la version d'alpine que nous utilisions. Je l'ai mis à jour.

Avec ça je peux lancer le site en local avec docker-compose up

Copy link
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai fait quelques suggestions, j'ajoute les points suivants:

  • On gagnerait à ajouter dans le fichier la note suivante: la base antoinede/transport_data_gouv_dev_database est une base ancienne, on pourrait à terme migrer à un "Postgres/postgis" bien à jour (patché etc), et avoir une procédure qui récupère le backup ou qui permet de donner le chemin vers le backup.
  • Il est nécessaire de créer un fichier .env pour que ça démarre. Le readme.md mentionne .envrc.example, mais ce fichier n'existe plus.

Dockerfile.dev Show resolved Hide resolved
Dockerfile.dev Show resolved Hide resolved
Dockerfile.dev Show resolved Hide resolved
@fchabouis
Copy link
Contributor Author

j'ai rajouté un fichier .env.example et j'ai mis à jour le readme

Copy link
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beau travail ; il faudra gérer le conflit avec #1785 juste après.

@fchabouis fchabouis merged commit 905fecd into master Sep 23, 2021
@thbar thbar deleted the fix-docker-compose branch October 7, 2021 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants