docker: add a Docker setup #2349
Conversation
Introduce a base Docker image and its companion docker-compose.yml page to explore containerizing the project. It's all, hopefully, straightforward but there's a small gotcha: the web service (which runs the Rails server) and the JS service (which runs the assets compilation) have to share the assets destination path. That's what the `assets` volume does. Everything else is pretty standard.
| @@ -1,4 +1,4 @@ | |||
| unless Rails.env.test? | |||
| if Rails.env.production? | |||
There was a problem hiding this comment.
seul obstacle pour le moment : AgentConnect qui essaie d'effectuer une connexion à un serveur externe même dans un environnement local
There was a problem hiding this comment.
Je suis d'accord on ne devrait pas avoir ce comportement en dev (ça ne devrait pas de poser de problème @Michaelvilleneuve ?)
There was a problem hiding this comment.
Effectivement on peut conditionner ça, de toute façon AgentConnect ne fonctionnera pas en dév
There was a problem hiding this comment.
pour info je crois que Romain a de toute façon rajouté cette condition dans sa PR donc ça finira pas être mergé avec ou sans cette PR
There was a problem hiding this comment.
Merci beaucoup @freesteph pour cette première brique très cool 🙏 !
Effectivement, ce serait top qu'avec une seule commande on puisse lancer rdv-i et rdv-sp en dev, surtout pour les nouveaux arrivants.
Ceci étant je ne sais pas si on aura beaucoup de temps à investir dessus prochainement, on a jusque là pas eu trop de soucis à faire tourner les applis sans docker et il n'est pas prévu à moyen terme de l'utiliser pour le déploiement.
| @@ -1,4 +1,4 @@ | |||
| unless Rails.env.test? | |||
| if Rails.env.production? | |||
There was a problem hiding this comment.
Je suis d'accord on ne devrait pas avoir ce comportement en dev (ça ne devrait pas de poser de problème @Michaelvilleneuve ?)
| @@ -0,0 +1,29 @@ | |||
| services: | |||
There was a problem hiding this comment.
(Dans le cas où on veut aller au-delà du POC et faire du dev faudra juste rajouter les workers ici)
Alors pour la question du déploiement je t'avoue que je n'y ai même pas pensé, c'est surtout le combo développement/onboarding très rapide + semi-documentation de l'infra (en tout cas de l'écosystème) que j'aime bien, après en déploiement Kube/containeurisé c'est pas encore très répandu dans la communauté donc beaucoup moins expert sur le sujet. Je vais quand même rajouter les workers comme tu l'as mentionné, et voir si j'arrive à lancer RDVI et RDVS côte à côte pour faire puisque sans surprise la Dockerisation côté RDVS se fait très facilement aussi (👏 à vous). On pourra reprendre un jour si vous avez le temps (et la motivation 😄 ), aucun soucis d'ici là merci d'avoir pris le temps de regarder. |
Oui c'est sûr que c'est un plus dans tous les cas ! Merci en tous cas c'est instructif pour nous ! |
Salut la team,
J'ai rencontré @Holist pendant le kickoff du FAST et on a discuté un peu de l'architecture de l'app et des potentiels sujets intéressants pour la suite, notamment celui d'offrir une containeurisation qui permette de mieux comprendre/isoler/démarrer (voir déployer un jour) l'application.
Je vous montre le bout de code qui m'a suffit à démarrer l'app, au moins qu'on puisse en discuter un peu et voir si c'est quelque chose qui vous intéresse.
PS : comme RDVI fonctionne en tandem avec RDVS il serait intéressant d'approfondir l'expérience en containeurisant les deux pour les lancer ensemble en quelque lignes de YAML...