Rendre vpn.env optionnel au lancement

O-clock-Dev / teleporter-compose

A containerized teleporter, to teleport student and teacher.

GNU Affero General Public License v3.0

3 stars 1 forks source link

Rendre vpn.env optionnel au lancement #27

Open clementgoclock opened 4 months ago

clementgoclock commented 4 months ago

Actuellement vpn.env est nécéssaire pour lancer le TP, il y est - vide - dans la code base pour permettre le lancement sans encombre au premier git clone

Cependant pour travailler c'est désagréable, il faut bien veiller à ne pas pousser son vpn.env quand on commit. Il faut aussi faire un copy sur un fichier déjà présent, etc.

Il faudrait que le vpn.env ne soit pas obligatoire pour lancer le compose.

Solution initiale : https://docs.docker.com/compose/environment-variables/set-environment-variables/

env_file:
  - path: ./default.env
    required: true # default
  - path: ./override.env
    required: false

Sensé marche dans la définition d'un service mais au niveau du compose (via l'include) docker compose explose (yep il pète, même pas un error ou warning, il pète). A voir si c'est un bug ou une feature.

clementgoclock commented 4 months ago

J'ai un gros doute sur le fait que ça puisse marcher dans le contexte d'un include mais dans le doute j'ai quand même ouvert un bug report https://github.com/docker/compose/issues/11772

clementgoclock commented 4 months ago

En workaround de ce required qu'on peut pas définir à une niveau plus élevé que chaque service :

Continuer avec un vpn.env vide dans la codebase avec un cp à chaque modification Git et depuis un autre fichier env qui lui correspond, à côté (et non suivi).
Générer un vpn.env à chaque lancement s'il n'existe pas (via un container dédié ou via le ct templating)
Gérer la configuration du VPN à travers l'interface web (d'une façon ou d'une autre) mais devra aussi nécéssiter une façon de reload les confs modifiées voir le compose

clementgoclock commented 4 months ago

J'ai un gros doute sur le fait que ça puisse marcher dans le contexte d'un include mais dans le doute j'ai quand même ouvert un bug report docker/compose#11772

Le crash/panic CLI a été fixé upstream (voir issue) mais la fonctionnalité n'est pas là, j'ai ouvert une feature request dans le doute. Ca pourra servir dans l'avenir si ça se débloque :)

clementgoclock commented 3 months ago

Actualisation le vpn.sample et trouver une soluce pour l'optionel du .env