Bug #112844
closedLe debian_config_common.py global de hobo prend le dessus sur les debian_config.py des briques (?)
0%
Description
Cela devrait être l’inverse (c’est le cas d’une installation Publik via les paquets debian), mais pour devinst spécifiquement, à l’occasion de #23494 je redéfinis dans le debian_config.py d’authentic la valeur de settings.SESSION_ENGINE pour y déclarer le nouveau moteur de session, et c’est quand même celle contenue dans le debian_config_commmon (le moteur de session mellon) qui reste utilisée.
Updated by Paul Marillonnet about 1 month ago
- Related to Développement #23494: Écran reprenant les connexions d'un utilisateur added
Updated by Paul Marillonnet about 1 month ago
- Related to Développement #112596: Laisser authentic déclarer son propre moteur de sessions added
Updated by Gael Pasgrimaud about 1 month ago
je dirai qu'on ne lit pas du tout debian_config.py dans devinst
Updated by Paul Marillonnet about 1 month ago
Ah, ok :D
Peut-être faudrait-il corriger cet état partiel des choses (où la config Debian commune est lue mais pas la config spécifique aux briques), que ce soit tout ou rien ?
Si oui, ce serait jouable dans les gabarits jinja2 spécifiques aux briques de dire de lire ~/src/<brique>/debian/debian_config.py ?
Updated by Gael Pasgrimaud about 1 month ago
Le problème c'est que dedans il y a des chemins en dur vers /var/lib
Ce qu'il faudrait c'est tenter de copier les fichiers depuis les différent debian/ pour les mettres dans ~/.config/publik/ puis remplacer les /var/lib/ par des chemins judicieux dans les fichiers copiés en utilisant la tache ansible lineinfile https://docs.ansible.com/projects/ansible/latest/collections/ansible/builtin/lineinfile_module.html
C'est un peu tricky
La solution de facilité c'est de reporter ton patch dans devinst (je crois que c'est plus ou moins ce qui est fait en l'état)
Updated by Emmanuel Cazenave about 1 month ago
Gael Pasgrimaud a écrit (#note-6):
Le problème c'est que dedans il y a des chemins en dur vers /var/lib
Oui tout à fait, j'étais tombé là dessus au tout début de devinst et ça avait donné #21643 et aussi #21723 où j'avais fait un petit nervous breakdown.
La solution de facilité c'est de reporter ton patch dans devinst (je crois que c'est plus ou moins ce qui est fait en l'état)
Encore oui et ça fait 8 ans qu'on se roule dans cette facilité :)
Updated by Benjamin Dauvergne about 1 month ago
Pour refaire chier comme à l'époque, je continue à penser qu'une variable d'environnement PUBLIK_BASEDIR par défaut à '/' et mis à $HOME/env/publik-env-py3/ par supervisor dans le cas devinst, ça aurait suffit sans changer aucun chemin relatif nulle part, la configuration serait dans ~/env/publik-env-py3/etc/*/settings.d/, etc... les debian_config.py pondus par devinst sont plus un souci qu'autre chose.
Updated by Paul Marillonnet about 1 month ago
- Status changed from Nouveau to Rejeté
Emmanuel Cazenave a écrit (#note-7):
La solution de facilité c'est de reporter ton patch dans devinst (je crois que c'est plus ou moins ce qui est fait en l'état)
Encore oui et ça fait 8 ans qu'on se roule dans cette facilité :)
Ok, ça n’est pas un besoin récurrent de ma part, juste en l’occurrence #23494 où authentic se voit doté de son propre moteur de session. Je ferai un patch ici au moment de la validation de ce dév.