-
Notifications
You must be signed in to change notification settings - Fork 161
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
Installation via docker-compose #6455
base: dev
Are you sure you want to change the base?
Conversation
Je n'ai pas réussi à joindre localhost. Après avoir fait
Je ne connais pas vraiment docker, donc je ne sais pas diagnostiquer plus que ça. Aussi, question subsidiaire pour @philippemilink : on préfère fusionner la présente PR après la finalisation de la nouvelle recherche ou après ? Il faudra adapter pour le nouveau moteur de recherche. J'ai une préférence pour après. C'est probablement plus facile d'adapter la présente PR que de rajouter encore une couche sur la PR de la recherche. |
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.
J'ai le même problème que @Arnaud-D concernant les messages Problem with dial
.
J'ai pas mal de remarques :
- ça rajoute quelques fichiers à la racine du dépôt qui sont spécifiques à Docker. Est-ce qu'on ne peut pas en déplacer un maximum dans un dossier dédié, par exemple
scripts/docker
? Je ne sais pas comment sont ensuite gérés les chemins relatifs... - concernant les Dockerfile, il me semble qu'une bonne pratique est de minimiser les appels à
RUN
, parce que ça multiplie le nombre d'étapes dans la création de l'image et ça fait des images plus grosses... - pour
back.Dockerfile
(surtout), ce qui m'embête, c'est que ça duplique (encore !) des commandes d'installation qui sont déjà ailleurs dans le dépôt. Donc quand on mettra le script d'installation à jour, il faudra aussi (penser à) mettre à jour les Dockerfile. Est-ce qu'il n'y a pas moyen de mieux factoriser ? - pour
docker.py
, j'imagine bien quelque chose du genrepour factoriser un maximum le code (pareil avecfrom .prod import DATABASES DATABASES["default"]["NAME"] = "zds_docker" # etc pour les autres champs à modifier
CACHES
)
Aussi, question subsidiaire pour @philippemilink : on préfère fusionner la présente PR après la finalisation de la nouvelle recherche ou après ? Il faudra adapter pour le nouveau moteur de recherche. J'ai une préférence pour après. C'est probablement plus facile d'adapter la présente PR que de rajouter encore une couche sur la PR de la recherche.
Ça m'est égal, ça ne me dérange pas d'adapter les fichiers Docker dans la PR sur la recherche. Je préfère peut-être même qu'on merge avec le moteur de recherche utilisé au moment du merge, ça évite d'avoir une PR qui dépend d'une autre PR (et on ne sait pas quand sera mergé la PR sur la recherche).
|
||
|
||
|
||
#COPY ./settings_docker.py /zds/zds/settings/docker.py | ||
#COPY ./service/zds-watchdog.sh /zds/zds-watchdog.sh | ||
#COPY ./service/zds-index.sh /zds/zds-index.sh |
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.
Si c'est des commentaires, à supprimer ?
) | ||
|
||
ZDS_APP["zmd"]["server"] = "http://zmd:27272" | ||
ZDS_APP["visual_changes"] = ["snow"] |
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.
Pourquoi on active la neige ?
"CONN_MAX_AGE": 600, | ||
"OPTIONS": { | ||
"charset": "utf8mb4", | ||
"init_command": "SET sql_mode='STRICT_TRANS_TABLES'", |
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.
Pourquoi on a cette ligne pour Docker, mais pas dans prod.py
?
@firm1 t'as prévu de rebosser dessus prochainement ? Si t'as pas le temps, pas de souci. C'est juste une histoire de suivi, on marquera la PR comme "en attente de reprise". |
L'objectif de ce changement est de rajouter le support de docker-compose comme moyen d'installer (et développer) zds en local via docker tout en évitant d'installer une quelconque bibliothèque sur notre système.
J'ai rajouté la documentation qui va avec pour pouvoir retrouver simplement les services déployés..
Fix #XXXXContrôle qualité
docker-compose up
http://localhost:8000
docker-compose up fixtures
http://localhost:8000
que vous pouvez vous pouvez vous connecter via le compte admin/admin