Projet

Général

Profil

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

Ajouté par Benjamin Dauvergne il y a 5 mois. Mis à jour il y a 5 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
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

Lié à Hobo - Development #83981: applification: propager les variables avec une valeur par défaut dès l'installation de l'applicationRejeté27 novembre 2023

Actions

Révisions associées

Révision edcd930d (diff)
Ajouté par Frédéric Péters il y a 5 mois

applications: add explicit call to notify after new variables (#83979)

Historique

#2

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).

#3

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 :

#4

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é
#5

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de En cours à Solution proposée
#6

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 :

#7

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 :

#8

Mis à jour par Transition automatique il y a 5 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a 3 mois

Automatic expiration

Formats disponibles : Atom PDF