Project

General

Profile

Development #24505

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

Added by Emmanuel Cazenave 12 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Start date:
13 Jun 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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 View (7.34 KB) Emmanuel Cazenave, 19 Nov 2018 03:21 PM

0001-store-settings-file-in-.config-publik-24505.patch View (7.33 KB) Emmanuel Cazenave, 27 Nov 2018 06:19 PM

Associated revisions

Revision 1d1e2b9d (diff)
Added by Emmanuel Cazenave about 1 month ago

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

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

History

#1 Updated by Emmanuel Cazenave 12 months ago

  • Description updated (diff)

#2 Updated by Emmanuel Cazenave 12 months ago

  • Description updated (diff)

#3 Updated by Emmanuel Cazenave 6 months ago

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 Updated by Paul Marillonnet 6 months ago

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

#6 Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution proposée to Solution validée
  • Assignee set to Emmanuel Cazenave

Ack.

#7 Updated by Emmanuel Cazenave about 1 month ago

  • Status changed from Solution validée to 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.

Also available in: Atom PDF