Autre #6711
Visibilité des services déployés dans Combo
0%
Description
Pour le moment il y a dans combo un paramétrage COMBO_WCS_SITES :
"COMBO_WCS_SITES": {
"default": {
"url": "https://demo.dev.au-quotidien.com/",
"title": "Demo",
"secret": "a",
"orig": "b"
}
Ça serait bien qu'un dictionnaire de services existe automatiquement selon le hobo.json.
{'wcs': ← service_type
{'eservices': ← slug service
{'url': base_url, ...
→
- modifier combo pour lire un settings de ce format (quel nom ? KNOWN_SERVICES ?)
- ajouter un middleware pour poser ça dans les settings.
Fichiers
Historique
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
C'est problématique de le mettre dans le middleware de template vars (et dans TEMPLATE_VARS aussi) ?
On l'aura partout (on ne le sait pas encore mais ça pourrait servir ailleurs, et ça me parait évident que ça servira dans des templates).
tenant_settings.KNOWN_SERVICES = etc.. tennat_settings.TEMPLATE_VARS['known-services'] = settings.KNOWN_SERVICES
Mis à jour par Serghei Mihai il y a presque 9 ans
- Fichier 0001-deployed-services-declared-in-KNOWN_SERVICES-setting.patch 0001-deployed-services-declared-in-KNOWN_SERVICES-setting.patch ajouté
En partant de la structure de dico proposée par Fréd:
{'wcs': ← service_type
{'eservices': ← slug service
{'url': base_url, ...
je verrais combo calculer son COMBO_WCS_SITES à partir du KNOWN_SERVICES et charge à hobo de le remplir.
Mis à jour par Frédéric Péters il y a presque 9 ans
Attention, l'idée côté Combo, c'est "modifier combo pour lire un settings de ce format", et non pas "générer le format actuel de COMBO_WCS_SITES à partir d'un autre settings".
Mis à jour par Serghei Mihai il y a presque 9 ans
- Fichier 0001-deployed-services-declared-in-KNOWN_SERVICES-setting.patch 0001-deployed-services-declared-in-KNOWN_SERVICES-setting.patch ajouté
- Fichier 0001-settings-deployed-services-exposed-in-KNOWN_SERVICES.patch 0001-settings-deployed-services-exposed-in-KNOWN_SERVICES.patch ajouté
Ok, je cherche les infos directement dans settings.KNOWN_SERVICES.
Les orig et secret dans KNOWN_SERVICES sont ceux du service en cours.
Mis à jour par Frédéric Péters il y a presque 9 ans
C'est bien simple, tout ce qui s'éloigne d'un bête et méchante remplacement de settings.COMBO_WCS_SITES par un settings.KNOWN_SERVICES.get('wcs', {}) (ou autre du même genre) complique la relecture et peut amener des soucis, que je préférerais éviter.
Il y a des changements de comportement flagrants :
- if len(settings.COMBO_WCS_SITES) == 1: - label = title - else: - label = '%s : %s' % (site_title, title) + label = '%s : %s' % (site_title, title)
et la modif au debian/debian_config.py devrait être séparée.
L'autre patch ne reprend pas du tout les considérations sur les clés mentionnées dans #7192, autant omettre tout à fait 'secret' et 'orig' ici, du coup.
Mis à jour par Serghei Mihai il y a presque 9 ans
- Fichier 0001-deployed-services-declared-in-KNOWN_SERVICES-setting.patch 0001-deployed-services-declared-in-KNOWN_SERVICES-setting.patch ajouté
- Fichier 0002-known-services-set-by-hobo.patch 0002-known-services-set-by-hobo.patch ajouté
- Fichier 0001-settings-deployed-services-exposed-in-KNOWN_SERVICES.patch ajouté
Patchs séparés pour combo.
Le secret est calculé dans hobo.
Mis à jour par Thomas Noël il y a presque 9 ans
(note sur 0002: ne pas le pousser ; oublions le packaging pour l'instant, surtout que ça sera plutôt dans /usr/lib/hobo/debian_config_common.py pour que tout le CoT dispose du dico)
0001-settings-deployed : pas ok pour moi, orig n'est pas domain
Mis à jour par Serghei Mihai il y a presque 9 ans
- Fichier
0001-settings-deployed-services-exposed-in-KNOWN_SERVICES.patchsupprimé
Mis à jour par Serghei Mihai il y a presque 9 ans
- Fichier 0001-settings-deployed-services-exposed-in-KNOWN_SERVICES.patch 0001-settings-deployed-services-exposed-in-KNOWN_SERVICES.patch ajouté
Corrigé, merci
Mis à jour par Frédéric Péters il y a presque 9 ans
Mis à jour par Serghei Mihai il y a presque 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
Appliqué par les commits:
- hobo: 92de406065cdc65a4a0d63ebdff5db34acfe8b81 * combo: 8bbd58a7ab8fba492987fb5551ffc33689c540a7
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-debian_config_common.py-add-KnownServices-tenant-set.patch 0001-debian_config_common.py-add-KnownServices-tenant-set.patch ajouté
- Statut changé de Résolu (à déployer) à Information nécessaire
Pour finir ce sujet, je propose cet énorme et magnifique patch à hobo qui installe hobo.multitenant.settings_loaders.KnownServices sur tous les composants Django de Publik
Mis à jour par Thomas Noël il y a presque 9 ans
- Statut changé de Information nécessaire à En cours
Mis à jour par Thomas Noël il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
Voilà. Ticket terminé pour de bon, je pense (testé sur vincennes.combo.dev.eo.org, tout ça marche très bien)
commit f6596d73384914f5dcf152de6bd619a702a20321
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed May 20 18:52:59 2015 +0200
debian_config_common.py: add KnownServices tenant settings (#6711)
Mis à jour par Serghei Mihai il y a environ 7 ans
- Statut changé de Résolu (à déployer) à Fermé
- Patch proposed mis à Non