Projet

Général

Profil

Development #24505

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

Ajouté par Emmanuel Cazenave il y a 8 mois. Mis à jour il y a 3 mois.

Statut:
Solution proposée
Priorité:
Normal
Assigné à:
-
Début:
13 juin 2018
Echéance:
% réalisé:

0%

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 Voir (7,34 ko) Emmanuel Cazenave, 19 nov. 2018 15:21

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

Historique

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

  • Description mis à jour (diff)

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

  • Description mis à jour (diff)

#3 Mis à jour par Emmanuel Cazenave il y a 3 mois

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 3 mois

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

Formats disponibles : Atom PDF