Skip to content
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

Rendre vpn.env optionnel au lancement #27

Open
clementgoclock opened this issue Apr 24, 2024 · 4 comments
Open

Rendre vpn.env optionnel au lancement #27

clementgoclock opened this issue Apr 24, 2024 · 4 comments
Labels
bug Something isn't working
Milestone

Comments

@clementgoclock
Copy link
Contributor

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
Copy link
Contributor Author

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

@clementgoclock
Copy link
Contributor Author

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
Copy link
Contributor Author

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 clementgoclock changed the title fix: vpn.env obligatoire pour lancer le teleporter Rendre vpn.env optionnel au lancement May 2, 2024
@clementgoclock clementgoclock added the bug Something isn't working label May 2, 2024
@clementgoclock clementgoclock added this to the v1 milestone May 28, 2024
@clementgoclock
Copy link
Contributor Author

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

@profy12 profy12 modified the milestones: v1, v2 Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In Progress
Development

No branches or pull requests

2 participants