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

[WIP] Ajoute configuration de développement pour docker-compose #4430

Closed
wants to merge 8 commits into
base: dev
from

Conversation

Projects
None yet
9 participants
@motet-a
Member

motet-a commented Aug 2, 2017

Les images se basent sur Debian et pas sur Alpine suite à quelques
problèmes de paquets binaires pas distribués pour musl.

Le conteneur du frontend a un accès en lecture seule aux sources, dans
le dossier assets/. Il fait tourner uniquement le processus de Gulp,
qui observe tous les fichiers et met à jour un volume monté sur
dist/ dès le moindre changement.

Ce même volume dist/ est monté en lecture seule dans le conteneur du
backend, qui fait tourner le processus de Django. Le dossier zds/
est également partagé en lecture seule.

La BDD, les fichiers des tutos et ceux des utilisateurs sont dans un
volume nommé, ils persistent.

Il n’y pas de LaTeX.

Il va falloir aussi rajouter un service pour le serveur de zmarkdown par la suite.


Ajoute clem, script pour utiliser docker-compose facilement

Je pense que c’est l’idéal pour des gens qui veulent débugger
rapidement un bout de JS sans avoir à lire un tas de documentation
sur comment se servir de Docker.

Ajoute également une petite page de documentation là-dessus.


Notez que j’utilise tout ceci depuis le début, je n’ai d’ailleurs jamais installé ZdS en “dur” sur mon PC. Cette PR a beaucoup évoluée depuis que je l’ai ouverte, il y a maintenant clem, de la documentation, des images plus légères et un support d’Elasticsearch.


  • Il y a un petit problème quand on arrête clem avec un Ctrl+C. Ça s’arrête mais docker-compose down reste attaché bizarrement au terminal.

Contrôle qualité

Vous devez installer Docker et docker-compose.

  1. Lancer ./clem up et attendez (longtemps)
  2. http://localhost:8000/
  3. Il n’y a pas de troisème étape.

Consultez la documentation (ou clem --help) et testez le reste (Elasticsearch, fixtures, tests unitaires, …).

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 2, 2017

Member

Ping @sandhose qui avait déjà un peu joué avec ça :)

Member

pierre-24 commented Aug 2, 2017

Ping @sandhose qui avait déjà un peu joué avec ça :)

@DevHugo

DevHugo approved these changes Aug 2, 2017

Manque quand même un minimum de documentation, au moins une procédure d'installation comme on a fait pour le back ou le front.

Show outdated Hide outdated docker-compose.yml Outdated
@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 2, 2017

Member

@DevHugo Je suis tout à fait d’accord, il manque de la documentation. Mais je pense que c’est encore un peu trop top pour en écrire vu que pour l’instant, c’est surtout un bidouillage qui ne sert qu’à moi et que l’arrivée de zmarkdown va tout chambouler.

Member

motet-a commented Aug 2, 2017

@DevHugo Je suis tout à fait d’accord, il manque de la documentation. Mais je pense que c’est encore un peu trop top pour en écrire vu que pour l’instant, c’est surtout un bidouillage qui ne sert qu’à moi et que l’arrivée de zmarkdown va tout chambouler.

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 2, 2017

Member

Pas d'accord. On nous a repproché par le passé la lourdeur de notre procédure d'installation, et c'est une solution possible. Même si ton alternative est "expérimentale", je pense que ça vaudrait la peine de la documenter, quitte à noter "expérimental" :p

(puis au pire, tu fait une PR sur la branche du nouveau zmd)

Member

pierre-24 commented Aug 2, 2017

Pas d'accord. On nous a repproché par le passé la lourdeur de notre procédure d'installation, et c'est une solution possible. Même si ton alternative est "expérimentale", je pense que ça vaudrait la peine de la documenter, quitte à noter "expérimental" :p

(puis au pire, tu fait une PR sur la branche du nouveau zmd)

@motet-a motet-a changed the title from Ajoute configuation de développement pour docker-compose to [WIP] Ajoute configuation de développement pour docker-compose Aug 2, 2017

Show outdated Hide outdated Dockerfile_front Outdated
Show outdated Hide outdated Dockerfile_back Outdated
Show outdated Hide outdated scripts/docker_dev_entrypoint.sh Outdated
@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Aug 2, 2017

Contributor

(puis au pire, tu fait une PR sur la branche du nouveau zmd)

ou un sujet sur le forum. Je suis entièrement d'accord avec pierre-24, on ne pourra pas merger sans documentation.

Question: du coup, qui va builder les images, le développer, dans ton cas. As-tu mesurer les avantages et inconvénients de ta solution par-rapport un build côté travis et téléchargement ?

Contributor

DevHugo commented Aug 2, 2017

(puis au pire, tu fait une PR sur la branche du nouveau zmd)

ou un sujet sur le forum. Je suis entièrement d'accord avec pierre-24, on ne pourra pas merger sans documentation.

Question: du coup, qui va builder les images, le développer, dans ton cas. As-tu mesurer les avantages et inconvénients de ta solution par-rapport un build côté travis et téléchargement ?

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 2, 2017

Member

Question: du coup, qui va builder les images, le développer, dans ton cas. As-tu mesurer les avantages et inconvénients de ta solution par-rapport un build côté travis et téléchargement ?

Même si ça serait assez intéréssant d'avoir par exemple une image pour chaque PR, j'ai peur que comme on part sur du "debian bien lourd", l'image sois pas légère. Mais je parle sans savoir ^^

Member

pierre-24 commented Aug 2, 2017

Question: du coup, qui va builder les images, le développer, dans ton cas. As-tu mesurer les avantages et inconvénients de ta solution par-rapport un build côté travis et téléchargement ?

Même si ça serait assez intéréssant d'avoir par exemple une image pour chaque PR, j'ai peur que comme on part sur du "debian bien lourd", l'image sois pas légère. Mais je parle sans savoir ^^

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Aug 2, 2017

Contributor

Même si ça serait assez intéréssant d'avoir par exemple une image pour chaque PR, j'ai peur que comme on part sur du "debian bien lourd", l'image sois pas légère. Mais je parle sans savoir ^^

Je pose la question, c'est tout ^^, ma question n'est pas orienté.

Contributor

DevHugo commented Aug 2, 2017

Même si ça serait assez intéréssant d'avoir par exemple une image pour chaque PR, j'ai peur que comme on part sur du "debian bien lourd", l'image sois pas légère. Mais je parle sans savoir ^^

Je pose la question, c'est tout ^^, ma question n'est pas orienté.

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 2, 2017

Member

Merci pour vos reviews. J’ai rajouté [WIP] au titre de la PR, je suis en train de m’occupper du problème de documentation et surtout de simplicité d’utilisation du bazar.

Concernant le poids des images de Debian, c’est tout relatif. Compressée, l’image de Debian 8 ne fait que 50 Mio, et ça monte vers 100 Mio décompressé. C’est juste que les images d’Alpine sont minuscules.

Member

motet-a commented Aug 2, 2017

Merci pour vos reviews. J’ai rajouté [WIP] au titre de la PR, je suis en train de m’occupper du problème de documentation et surtout de simplicité d’utilisation du bazar.

Concernant le poids des images de Debian, c’est tout relatif. Compressée, l’image de Debian 8 ne fait que 50 Mio, et ça monte vers 100 Mio décompressé. C’est juste que les images d’Alpine sont minuscules.

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Aug 2, 2017

Contributor

Concernant le poids des images de Debian, c’est tout relatif. Compressée, l’image de Debian 8 ne fait que 50 Mio, et ça monte vers 100 Mio décompressé. C’est juste que les images d’Alpine sont minuscules.

On vera sur combien ça joue sur le poids final de ton image, si c'est négligeable ou pas. Combien de temps prends ton image à build ? as tu vérifié la sécurité de ton image avec les outils associés sur docker hub ?

Contributor

DevHugo commented Aug 2, 2017

Concernant le poids des images de Debian, c’est tout relatif. Compressée, l’image de Debian 8 ne fait que 50 Mio, et ça monte vers 100 Mio décompressé. C’est juste que les images d’Alpine sont minuscules.

On vera sur combien ça joue sur le poids final de ton image, si c'est négligeable ou pas. Combien de temps prends ton image à build ? as tu vérifié la sécurité de ton image avec les outils associés sur docker hub ?

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 2, 2017

Member

On vera sur combien ça joue sur le poids final de ton image, si
c'est négligeable ou pas.

Ça joue assez peu sur la taille des images finales. Actuellement,
celle du backend fait 1 Gio et celle du frontend 755 Mio. C’est
pas tout petit mais je ne sais pas si ça vaut la peine de faire
plus d’effort pour réduire ça.

Combien de temps prends ton image à build ?

Chez moi, ça prends un petit bout de temps parce que je suis en
ADSL. Mais personellement, je préfère patienter 15 minutes qu’une
image build en faisant autre chose que de passer 30 minutes à
lire de la doc pour faire tourner ce site chez moi :-)

As tu vérifié la sécurité de ton image avec les outils associés
sur docker hub ?

Non. Mais étant donné que ce n’est pas pour la production, je
doute que ce soit important.

Member

motet-a commented Aug 2, 2017

On vera sur combien ça joue sur le poids final de ton image, si
c'est négligeable ou pas.

Ça joue assez peu sur la taille des images finales. Actuellement,
celle du backend fait 1 Gio et celle du frontend 755 Mio. C’est
pas tout petit mais je ne sais pas si ça vaut la peine de faire
plus d’effort pour réduire ça.

Combien de temps prends ton image à build ?

Chez moi, ça prends un petit bout de temps parce que je suis en
ADSL. Mais personellement, je préfère patienter 15 minutes qu’une
image build en faisant autre chose que de passer 30 minutes à
lire de la doc pour faire tourner ce site chez moi :-)

As tu vérifié la sécurité de ton image avec les outils associés
sur docker hub ?

Non. Mais étant donné que ce n’est pas pour la production, je
doute que ce soit important.

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Aug 2, 2017

Contributor

Ça joue assez peu sur la taille des images finales. Actuellement,
celle du backend fait 1 Gio et celle du frontend 755 Mio. C’est
pas tout petit mais je ne sais pas si ça vaut la peine de faire
plus d’effort pour réduire ça.

ça me parait beaucoup, les images que j'avais build à l'époque (y'a un an) :

Peut-être qu'on parle pas de la même chose.

Chez moi, ça prends un petit bout de temps parce que je suis en
ADSL. Mais personellement, je préfère patienter 15 minutes qu’une
image build en faisant autre chose que de passer 30 minutes à
lire de la doc pour faire tourner ce site chez moi :-)

Le tout est de savoir étape par étape le temps de build qu'on puisse optimiser tout cela, par exemple, comment s'organise t-on pour faire le npm install, qui le fait et pourquoi.
Je ne conteste pas le choix de docker mais je pose des questions pour qu'on puisse au mieux cet outil.

Non. Mais étant donné que ce n’est pas pour la production, je
doute que ce soit important.

C'est vrai, n'empêche qu'on peut appliquer des principes de sécurités simple et peut couteux en terme de développement, par-exemple, as tu besoin d'être root sur ton container ?
C'est pas indispensable, c'est juste pour soulever la question.

Contributor

DevHugo commented Aug 2, 2017

Ça joue assez peu sur la taille des images finales. Actuellement,
celle du backend fait 1 Gio et celle du frontend 755 Mio. C’est
pas tout petit mais je ne sais pas si ça vaut la peine de faire
plus d’effort pour réduire ça.

ça me parait beaucoup, les images que j'avais build à l'époque (y'a un an) :

Peut-être qu'on parle pas de la même chose.

Chez moi, ça prends un petit bout de temps parce que je suis en
ADSL. Mais personellement, je préfère patienter 15 minutes qu’une
image build en faisant autre chose que de passer 30 minutes à
lire de la doc pour faire tourner ce site chez moi :-)

Le tout est de savoir étape par étape le temps de build qu'on puisse optimiser tout cela, par exemple, comment s'organise t-on pour faire le npm install, qui le fait et pourquoi.
Je ne conteste pas le choix de docker mais je pose des questions pour qu'on puisse au mieux cet outil.

Non. Mais étant donné que ce n’est pas pour la production, je
doute que ce soit important.

C'est vrai, n'empêche qu'on peut appliquer des principes de sécurités simple et peut couteux en terme de développement, par-exemple, as tu besoin d'être root sur ton container ?
C'est pas indispensable, c'est juste pour soulever la question.

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 2, 2017

Member

ça me parait beaucoup, les images que j'avais build à l'époque (y'a un an)

Les tailles des images indiquées sur le Docker Hub sont compressées. Décompressée, arhenion/zds-docker:front-latest fait 441 Mio, contrairement à la mienne qui fait 755 Mio. La mienne se base sur node:8 qui se base sur du buildpack-deps, elle inclus donc GCC et compagnie. Je crois que j’avais des problèmes avec node:8-slim parce que certains packets de Node.js étaient dépassés et ne fournissaient pas de binaires déjà compilés à télécharger. Mais maintenant que les deps sont à jour, il faut que j’essaye de changer ça. Si on se base sur du node:slim, le temps de construction devrait être vraiment réduit vu qu’il n’y aura aucune compilation lors du npm install.

J’ai pas regardé le backend en détails. Ça serait bien d’avoir un retour de quelqu’un qui connait bien Django. Je connais mieux Node.js que Django 🙃

Le tout est de savoir étape par étape le temps de build qu'on puisse optimiser tout cela

J’ai pas une connection à internet très représentative actuellement et je pense qu’il faudrait déjà bien s’assurer que ça marche correctement, de corriger les problèmes évidents et d’écrire la doc avant d’optimiser.

C'est vrai, n'empêche qu'on peut appliquer des principes de sécurités simple et peut couteux en terme de développement, par-exemple, as tu besoin d'être root sur ton container ?

Je ne suis pas spécialiste mais je ne vois pas l’intérêt de ne pas être root dans ce cas. Les conteneurs Docker sont complètement isolés entre eux et de l’hôte. Si je suis root dans un conteneur, j’ai un uid invalide par rapport à l’hôte.

Member

motet-a commented Aug 2, 2017

ça me parait beaucoup, les images que j'avais build à l'époque (y'a un an)

Les tailles des images indiquées sur le Docker Hub sont compressées. Décompressée, arhenion/zds-docker:front-latest fait 441 Mio, contrairement à la mienne qui fait 755 Mio. La mienne se base sur node:8 qui se base sur du buildpack-deps, elle inclus donc GCC et compagnie. Je crois que j’avais des problèmes avec node:8-slim parce que certains packets de Node.js étaient dépassés et ne fournissaient pas de binaires déjà compilés à télécharger. Mais maintenant que les deps sont à jour, il faut que j’essaye de changer ça. Si on se base sur du node:slim, le temps de construction devrait être vraiment réduit vu qu’il n’y aura aucune compilation lors du npm install.

J’ai pas regardé le backend en détails. Ça serait bien d’avoir un retour de quelqu’un qui connait bien Django. Je connais mieux Node.js que Django 🙃

Le tout est de savoir étape par étape le temps de build qu'on puisse optimiser tout cela

J’ai pas une connection à internet très représentative actuellement et je pense qu’il faudrait déjà bien s’assurer que ça marche correctement, de corriger les problèmes évidents et d’écrire la doc avant d’optimiser.

C'est vrai, n'empêche qu'on peut appliquer des principes de sécurités simple et peut couteux en terme de développement, par-exemple, as tu besoin d'être root sur ton container ?

Je ne suis pas spécialiste mais je ne vois pas l’intérêt de ne pas être root dans ce cas. Les conteneurs Docker sont complètement isolés entre eux et de l’hôte. Si je suis root dans un conteneur, j’ai un uid invalide par rapport à l’hôte.

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 2, 2017

Member

Btw, arhenion/zds-docker:back-latest fait curieusement que 379 Mio décompressée. Il faut que j’essaye cette image.

Member

motet-a commented Aug 2, 2017

Btw, arhenion/zds-docker:back-latest fait curieusement que 379 Mio décompressée. Il faut que j’essaye cette image.

Show outdated Hide outdated clem Outdated

@motet-a motet-a changed the title from [WIP] Ajoute configuation de développement pour docker-compose to [WIP] Ajoute configuration de développement pour docker-compose Aug 2, 2017

Show outdated Hide outdated doc/source/index.rst Outdated
Show outdated Hide outdated doc/source/docker.rst Outdated
Show outdated Hide outdated doc/source/docker.rst Outdated
@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 3, 2017

Member

Bon, il y a un gros problème avec les settings en fait, je suis en train de régler ça sur la #4438.

Member

motet-a commented Aug 3, 2017

Bon, il y a un gros problème avec les settings en fait, je suis en train de régler ça sur la #4438.

Show outdated Hide outdated doc/source/docker.rst Outdated
@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 3, 2017

Member

Bon, ne prenez pas peur, il y a dans les commits des choses qui n’ont rien à voir avec cette PR mais ils seront enlevés. Il faut juste ne pas merge cette PR pour l’instant.

Member

motet-a commented Aug 3, 2017

Bon, ne prenez pas peur, il y a dans les commits des choses qui n’ont rien à voir avec cette PR mais ils seront enlevés. Il faut juste ne pas merge cette PR pour l’instant.

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 6, 2017

Member

Tous les commits sans rapport sur lesquels était basée cette PR ont étés merge dans dev !

Member

motet-a commented Aug 6, 2017

Tous les commits sans rapport sur lesquels était basée cette PR ont étés merge dans dev !

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Aug 7, 2017

Member

Je viens de mettre à jour la documentation que vous pouvez voir en ligne ici.

Member

motet-a commented Aug 7, 2017

Je viens de mettre à jour la documentation que vous pouvez voir en ligne ici.

@motet-a motet-a changed the title from [WIP] Ajoute configuration de développement pour docker-compose to Ajoute configuration de développement pour docker-compose Aug 9, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Aug 16, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Aug 16, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Aug 16, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Aug 16, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 16, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 21, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Nov 21, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 29, 2017

@AmarOk1412

This comment has been minimized.

Show comment
Hide comment
@AmarOk1412

AmarOk1412 Jan 3, 2018

Member

Pour le moment j'obtiens :

temp.txt

je regarderais plus tard

Member

AmarOk1412 commented Jan 3, 2018

Pour le moment j'obtiens :

temp.txt

je regarderais plus tard

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 8, 2018

Coverage Status

Coverage decreased (-0.07%) to 89.498% when pulling 9392873 on motet-a:docker-dev into a815bff on zestedesavoir:dev.

coveralls commented Jan 8, 2018

Coverage Status

Coverage decreased (-0.07%) to 89.498% when pulling 9392873 on motet-a:docker-dev into a815bff on zestedesavoir:dev.

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

motet-a and others added some commits Aug 1, 2017

Ajoute configuation de développement pour docker-compose
Les images se basent sur Debian et pas sur Alpine suite à quelques
problèmes de paquets binaires pas distribués pour musl.

Le conteneur du frontend a un accès en lecture seule aux sources, dans
le dossier `assets/`. Il fait tourner uniquement le processus de Gulp,
qui observe tous les fichiers et met à jour un volume monté sur
`dist/` dès le moindre changement.

Ce même volume `dist/` est monté en lecture seule dans le conteneur du
backend, qui fait tourner le processus de Django. Le dossier `zds/`
est également partagé en lecture seule.

Pendant que ça tourne, les fixtures peuvent être appliquées comme ça :

    docker-compose exec back \
        python manage.py loaddata \
            --settings zds.settings_docker_dev \
            fixtures/users.yaml

Et lorsque tout est arrêté, les tests peuvent être lancés comme ça :

    docker-compose run back \
        python manage.py test \
            --settings zds.settings_docker_test

Et pour lint le backend :

    docker-compose exec back \
        make lint-back

La BDD, les fichiers des tutos et ceux des utilisateurs sont dans un
volume nommé, ils persistent.
Ajoute `clem`, script pour utiliser docker-compose facilement
Bon, j’étais un peu en panne d’inspiration pour le nom, du coup il
s’appelle `clem` pour l’instant. C’est un petit script compatible avec
Python 2 et Python 3 qui lance docker-compose.

Évidemment, c’est très limité. Mais je pense que c’est l’idéal pour
des gens qui veulent débugger rapidement un bout de JS sans avoir à
lire un tas de documentation sur comment se servir de Docker.

Ajoute également une petite page de documentation là-dessus.
Permet de lancer les tests avec selenium et Firefox
Utilise Xvfb un peu comme Travis.
Préférrer DJANGO_SETTINGS_MODULE à --settings
Ça fait moins de répétitions.
docker-compose: fix les options pour monter les volumes
L'utilisation de SELinux pour Docker (active sur les distributions type RHEL)
empechera l'utilisation correcte des volumes. Le flag :z doit-etre mis sur les
fichiers utilises par de multiples containers (ou :Z si un seul container
utilise le fichier).
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 19, 2018

Coverage Status

Coverage decreased (-0.1%) to 89.5% when pulling cac591c on motet-a:docker-dev into e3782ce on zestedesavoir:dev.

coveralls commented Jan 19, 2018

Coverage Status

Coverage decreased (-0.1%) to 89.5% when pulling cac591c on motet-a:docker-dev into e3782ce on zestedesavoir:dev.

@@ -0,0 +1,27 @@
FROM python:3-stretch

This comment has been minimized.

@Eskimon

Eskimon Apr 30, 2018

Member

Faudrait sûrement préciser une version similaire à celle de la prod', pour se rapprocher des conditions de prod'.

@Eskimon

Eskimon Apr 30, 2018

Member

Faudrait sûrement préciser une version similaire à celle de la prod', pour se rapprocher des conditions de prod'.

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon Apr 30, 2018

Member

Je trouve cette PR intéressante, ca serait cool qu'elle voit le jour. Il y aurait moyen d'ajouter un service mysql pour avoir une bdd proche de celle de la prod' (et tester ainsi plus facilement les cas limites liés aux différences mysql/sqlite) ?

Member

Eskimon commented Apr 30, 2018

Je trouve cette PR intéressante, ca serait cool qu'elle voit le jour. Il y aurait moyen d'ajouter un service mysql pour avoir une bdd proche de celle de la prod' (et tester ainsi plus facilement les cas limites liés aux différences mysql/sqlite) ?

@pierre-24 pierre-24 changed the title from Ajoute configuration de développement pour docker-compose to [WIP] Ajoute configuration de développement pour docker-compose Aug 4, 2018

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 Aug 16, 2018

Contributor

Bonjour,

Vu le nombre de PR en cours, j'aimerai faire un peu de QA pour vider un peu la stack. C'est la raison qui m'amène sur cette PR. Je ne sais pas si @motet-a est encore actif, car j'aimerai bien reprendre sa PR pour l'amener au bout. J'ai noté les éléments suivants qui sont a faire :

  • passer sur le nouveau zmd (ce n'est pas encore le cas dans cette PR)
  • passer sur mysql au lieu de sqlite comme le suggère @Eskimon ci-dessus
  • mettre à jour la documentation

Voilà un peu le plan que j'ai. ça vous convient ?

Contributor

firm1 commented Aug 16, 2018

Bonjour,

Vu le nombre de PR en cours, j'aimerai faire un peu de QA pour vider un peu la stack. C'est la raison qui m'amène sur cette PR. Je ne sais pas si @motet-a est encore actif, car j'aimerai bien reprendre sa PR pour l'amener au bout. J'ai noté les éléments suivants qui sont a faire :

  • passer sur le nouveau zmd (ce n'est pas encore le cas dans cette PR)
  • passer sur mysql au lieu de sqlite comme le suggère @Eskimon ci-dessus
  • mettre à jour la documentation

Voilà un peu le plan que j'ai. ça vous convient ?

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 Aug 20, 2018

Contributor

Je ferme ici, on reprend les choses là bas.

Contributor

firm1 commented Aug 20, 2018

Je ferme ici, on reprend les choses là bas.

@firm1 firm1 closed this Aug 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment