Bug #83979
applification, lors de l'installation d'une application avec un paramètre, la création de la Variable ne semble pas provoquer de message hobo-deploy
Début:
27 novembre 2023
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Description
Cf. #83904
Reproduit par Mike sur une application minimale, le code suivant qui se déroule pendant le déploiement de l'application ne semble pas exécuter le signal post_save sur l'instance de Variable :
def deploy_parameters(self, bundle): tar_io = io.BytesIO(bundle) with tarfile.open(fileobj=tar_io) as tar: manifest = json.loads(tar.extractfile('manifest.json').read().decode()) for parameter in manifest.get('parameters') or []: param, _ = Parameter.objects.get_or_create( name=parameter.get('name'), application=self.application ) param.label = parameter.get('label') param.default_value = parameter.get('default_value') param.save() variable, _ = Variable.objects.get_or_create( name=parameter.get('name'), defaults={'auto': True, 'value': parameter.get('default_value') or ''}, ) variable.label = parameter.get('label') variable.save()
Le signal post_save en question :
$ grep -B2 -A4 post_en hobo/deploy/signals.py @receiver(post_save) @receiver(post_delete) def post_environment_save(sender, instance, **kwargs): if not sender in [Variable, AttributeDefinition] + AVAILABLE_SERVICES: return tls.MUST_NOTIFY = True
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a 5 mois
- Assigné à mis à Frédéric Péters
Il y a un receiver(request_finished)
(et le déploiement n'a pas lieu dans un requête).
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Nouveau à En cours
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/hobo/pulls/91
- Titre : WIP: applications: add explicit call to notify after new variables (#83979)
- Modifications : https://git.entrouvert.org/entrouvert/hobo/pulls/91/files
Mis à jour par Frédéric Péters il y a 5 mois
- Lié à Development #83981: applification: propager les variables avec une valeur par défaut dès l'installation de l'application ajouté
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne (bdauvergne) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/hobo/pulls/91
- Titre : applications: add explicit call to notify after new variables (#83979)
- Modifications : https://git.entrouvert.org/entrouvert/hobo/pulls/91/files
Mis à jour par Transition automatique il y a 5 mois
- Statut changé de Résolu (à déployer) à Solution déployée
applications: add explicit call to notify after new variables (#83979)