Projet

Général

Profil

Development #24505

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

Ajouté par Emmanuel Cazenave il y a presque 6 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Emmanuel Cazenave
Version cible:
-
Début:
13 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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.

Fichiers

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

Révisions associées

Révision 1d1e2b9d (diff)
Ajouté par Emmanuel Cazenave il y a environ 5 ans

store settings file in ~/.config/publik (#24505)

And allow additional user config in settings.d
sub directories.

Historique

#1

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

  • Description mis à jour (diff)
#2

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

  • Description mis à jour (diff)
#3

Mis à jour par Emmanuel Cazenave il y a plus de 5 ans

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 plus de 5 ans

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

#6

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée
  • Assigné à mis à Emmanuel Cazenave

Ack.

#7

Mis à jour par Emmanuel Cazenave il y a environ 5 ans

  • Statut changé de Solution validée à Solution déployée
commit 1d1e2b9dc78ba233066afa7d15aeac0480235e0d
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Fri Nov 16 08:28:53 2018 +0100

    store settings file in ~/.config/publik (#24505)

    And allow additional user config in settings.d
    sub directories.

Formats disponibles : Atom PDF