Docker: move config back to volume /var/lib/duniter and other minor changes

Main change:

• Set back configuration home to /var/lib/duniter/duniter_default. The volume /etc/duniter was created for the keyfile.

Minor changes:

• Merge duniter.sh wrapper into the entrypoint
• Install duniter into /duniter instead of /duniter/duniter
• Doc: fix commit naming convention example
• Entrypoint: run duniter with exec
• Add .dockerignore file

@librelois, voici ce que j'ai dans ma pile. Dis-moi si tu préfères que je scinde en plusieurs MR.

La suppression du volume /etc/duniter me semble prioritaire pour apaiser les esprits...

added 1 commit

• a7d0c161 - fix(docker): entrypoint - use jq merge operator

Compare with previous version

Drop volume /etc/duniter

@pini-gh non ce volume existait déjà avant, merci de le laisser. @sveyret l'avais créer afin que l'on puisse stocker le fichier trousseau séparément des données.

Il faut juste que l'usage de ce 2ème volume redevienne facultatif. Soit en remettant la conf dans le volume data, soit en proposant les 2 possibilités via une variable d'environnement (le comportement par défaut devant alors être l'usage d'un seul volume) :)

Every conf.json option is configurable from environment variable such as: DUNITER_CONF_=<value»

comme dit sur le forum, ça me semnble être une fausse bonne idée pour 2 raisons:

1. Ça ne fonctionnerai que pour le livrable docker
2. Ça risque de ne pas fonctionner pour une nouvelle instance vierge de Duniter

De plus, ça ne pourra pas fonctionner pour les modules rust (gva, future admin, etc).

Je préférerai gérer ça directement dans le code, d'autant que j'ai commencé en ce sens !

Dis-moi si tu préfères que je scinde en plusieurs MR.

Ça m'est égal, fait selon ce qui est le plus pratique pour toi :)

Il faut juste que l'usage de ce 2ème volume redevienne facultatif. Soit en remettant la conf dans le volume data, soit en proposant les 2 possibilités via une variable d'environnement (le comportement par défaut devant alors être l'usage d'un seul volume) :)

Dans ce cas, il est possible de le supprimer du Dockerfile. Cela n’empêchera pas celui qui veut de le déclarer à la création du conteneur (avec l’option --volume de docker run).

added 2 commits

• 9e3a9ecb - ref(docker): move back config to /var/lib/duniter
• a9bc88f7 - fix(doc): volume /etc/duniter is for the keyfile

Compare with previous version

changed title from Docker: drop volume /etc/duniter and allow setting conf.json from environment variables to Docker: move config back to volume /var/lib/duniter and other minor changes

changed the description

@librelois j'ai viré le commit relatif aux variables d'environnement, et conservé le volume /etc/duniter afin qu'il puisse continuer de servir pour le trousseau. Je maintiens toutefois le retour de la config dans le home Duniter.

Merge duniter.sh wrapper into the entrypoint

Profite en pour renommer le fichier en supprimant le préfixe docker-, on est déjà dans le dossier docker donc on sais que le script est pour docker.

Add .dockerignore file

tu peut y ajouter les dossiers :

.cargo
.github
.gitlab
gui
test

j'ai viré le commit relatif aux variables d'environnement, et conservé le volume /etc/duniter afin qu'il puisse continuer de servir pour le trousseau. Je maintiens toutefois le retour de la config dans le home Duniter.

Ok

added 6 commits

• 1426872f - build(docker): add .dockerignore file
• caaf6897 - fix(doc): commit naming conventions example
• 255bd02d - build(docker): entrypoint: run duniter with exec
• a05d320b - build(docker): install into /duniter
• 544f21e4 - ref(docker): move back config to /var/lib/duniter
• e88b5aa6 - fix(doc): volume /etc/duniter is for the keyfile

Compare with previous version

@pini-gh concernant les nom des commit: un fix est réservé à la correction d'un bug dans le code, ce type de commit ne doit pas être utilisé pour une modification de la doc, pour une modification de la doc, 2 possibilités :

1. Soit elle est intégrée dans le même commit que la modification de ce qui est documenté, c'est l'idéal.
2. Soit elle est dans un commit à part, qui sera donc toujours de type doc.

@librelois comme dit plus haut, je propose de retirer mon commit e88b5aa6.

Pour le caaf6897, est-ce que ceci irait ?

doc(dev): fix commit naming example

added 4 commits

• 23b11afa - doc(dev): fix commit naming example
• 8e200e5b - build(docker): entrypoint: run duniter with exec
• 3a6f4eaa - build(docker): install into /duniter
• c9062471 - ref(docker): move back config to /var/lib/duniter

Compare with previous version

added 1 commit

• 4a1ce6e8 - build(docker): WORKDIR /var/lib/duniter

Compare with previous version

changed the description

changed the description

J'avais brièvement poussé un commit pour changer le répertoire de travail par défaut en /var/lib/duniter, mais apparemment j'avais mal testé car en relançant ce matin ça ne marchait plus. Je suis donc revenu en arrière.

Pour le caaf6897, est-ce que ceci irait ?

Oui très bien :)

J'avais brièvement poussé un commit pour changer le répertoire de travail par défaut en /var/lib/duniter, mais apparemment j'avais mal testé car en relançant ce matin ça ne marchait plus. Je suis donc revenu en arrière.

Ok d'accord. Est ce que tu à fini ? J'attends que tu est fini pour pouvoir lancer la CI :)

@librelois : oui, j'ai terminé sur ce lot. Go :)

resolved all threads