Development #24505

Permettre à l'utilisateur d'écrire de la configuration additionnelle

Ajouté par Emmanuel Cazenave il y a 6 mois. Mis à jour il y a 15 jours.

Statut:Solution proposéeDébut:13 juin 2018
Priorité:NormalEchéance:
Assigné à:-% réalisé:

0%

Catégorie:-
Version cible:-
Patch proposed:Oui

Description

Un peu comme ce qui est fait sur nos packages debian, où les /etc/app/settings.py chargent des /etc/app/settings.d/*.py.

Dans publik devinst le fichier de configuration principal est ~/path/to/virtenv/conf/app_settings.py (chemin qui serait d'ailleurs peut-être changer, c'est pas terrible), mais ~/path/to/virtenv/ est supprimé si on exécute un clean.yml.

Donc on pourrait faire en sorte que ~/path/to/virtenv/conf/app_settings.py charge par exemple un ~/.config/publik-devinst/app_settings.py si il existe, que publik-devinst ne modifierais ni ne supprimerais jamais.

0001-store-settings-file-in-.config-publik-24505.patch Magnifier (7,34 ko) Emmanuel Cazenave, 19 nov. 2018 15:21

0001-store-settings-file-in-.config-publik-24505.patch Magnifier (7,33 ko) Emmanuel Cazenave, 27 nov. 2018 18:19

Historique

#1 Mis à jour par Emmanuel Cazenave il y a 6 mois

  • Description mis à jour (diff)

#2 Mis à jour par Emmanuel Cazenave il y a 6 mois

  • Description mis à jour (diff)

#3 Mis à jour par Emmanuel Cazenave il y a 23 jours

Voilà ce que ça donne pour l'arborescence des fichiers :

cazino@cazino-laptop:~/.config$ tree publik/
publik/
└── settings
    ├── authentic-multitenant
    │   ├── settings.d
    │   └── settings.py
    ├── bijoe
    │   ├── settings.d
    │   └── settings.py
    ├── chrono
    │   ├── settings.d
    │   └── settings.py
    ├── combo
    │   ├── settings.d
    │   └── settings.py
    ├── fargo
    │   ├── settings.d
    │   └── settings.py
    ├── hobo
    │   ├── settings.d
    │   └── settings.py
    ├── hobo-agent
    │   └── settings.py
    ├── passerelle
    │   ├── settings.d
    │   └── settings.py
    ├── wcs
    │   ├── settings.d
    │   ├── settings.py
    │   └── wcs.cfg
    └── welco
        ├── settings.d
        └── settings.py

Avec tous les settings.py qui seront écrasés lors d'une mise à jour mais pas les settings.d/*.

J'ai mis ça dans ~/.config/publik/settings dans l'idée qu'on pourra rajouter ensuite des ~/.config/publik/themes et autres, bref enlever du virtualenv tout ce qui n'a rien à y faire.

Pas réussi à être complètement DRY sur le patch parce que wcs et hobo-agent ont des particularités qui ne rentrent pas dans le rôle app-setup.

#4 Mis à jour par Paul Marillonnet il y a 15 jours

Est-ce que tu peux rebaser le patch maintenant que #28135 et #28231 sont dans master ?

Formats disponibles : Atom PDF