Development #60572
Stocker les infos sur soi même dans environment.models ::Hobo (pour pouvoir changer de nom de domaine)
0%
Description
Actuellement une instance d'hobo ne stocke pas les infos sur elle même dans environment.models::Hobo (ça sert uniquement dans le cas multi-collectivité, dans ce cas y sont stockés les infos sur les "autres" hobo).
Dans #58908, la mécanique introduite nécessite de stocker les anciens URL d'un service (pour que ce soit ensuite diffusé dans le hobo.json et que chaque brique puisse en tenir compte), du coup on veut qu'une instance d'hobo stocke les infos sur elle même pour pouvoir tirer partie de cette mécanique.
Préalable à #60566.
Fichiers
Demandes liées
Révisions associées
Revert "environment: store local hobo info in Hobo model (#60572)"
This reverts commit 00f897753b0bda2964c7ae564dc7a6073bea20bc.
Historique
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
- Lié à Bug #60566: Permettre de changer le nom de domaine d'hobo ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
- Lié à Support #57729: Migration d'instances de Publik : changement de nom de domaine ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
- Fichier 0001-environment-store-local-hobo-info-in-Hobo-model-6057.patch 0001-environment-store-local-hobo-info-in-Hobo-model-6057.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Pas compliqué, ajout d'une colonne 'emitter' sur le modèle Hobo pour discriminer et j'opte pour le get_or_create plutôt qu'une migration parce que ça fait un patch assez simple tandis que la migration s'annonçait galère (pour atteindre toutes les données dont on a besoin).
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution proposée à En cours
Relecture avec Thomas sur mes genoux.
Consensus pour :
- faire une vrai migration plutôt que se reposer sur un get_or_create
- renommer le champ 'emitter' en .... on ne sait pas, myself ?
Mis à jour par Thomas Noël il y a environ 2 ans
Emmanuel Cazenave a écrit :
- renommer le champ 'emitter' en .... on ne sait pas, myself ?
Comme on a déjà un truc qui s'appelle "local_hobo" dans le code, plutôt "local".
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-environment-store-local-hobo-info-in-Hobo-model-6057.patch 0001-environment-store-local-hobo-info-in-Hobo-model-6057.patch ajouté
- Statut changé de En cours à Solution proposée
Remarques prises en compte.
A note les précautions dans la migration parce que ça passe pas dans tous les tests d'y jouer avec avec les tenant (il y a des objets FakeTenant qui se balladent, ce genre de choses).
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Sur
local = models.BooleanField(default=False)
on pourrait mettre un help_text en guise de documentation/rappel, genre « help_text=_('True if this instance is the hobo tenant itself') » mais en l'écrivant je constate mes lacunes en anglais pour dire « Vrai si cette instance représente le tenant hobo lui-même »)
Ack avec ou sans ça.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
J'ai mis un commentaire pour ne pas s'obliger à traduire un truc qui n’apparaît nulle part :
# historically an hobo instance did not store information about itself, # it only stored info about other hobo's. # now we also store info about an instance on the instance itself, via local=True
commit 00f897753b0bda2964c7ae564dc7a6073bea20bc
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Thu Jan 13 17:56:23 2022 +0100
environment: store local hobo info in Hobo model (#60572)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Bug #62017: la création d'instance "Hobo" pour l'hobo lui-même aurait pété la barre latérale publik (?) ajouté
environment: store local hobo info in Hobo model (#60572)