Development #7201
hobo-deploy n'est pas lancé en cas de suppression d'un objet (variable, service, etc)
0%
Description
Actuellement la notification de l'agent (hobo-deploy) est faite uniquement sur post_save (cf hobo/deploy/signals.py).
Sans doute faudrait-il le faire sur pre_delete aussi.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Dupliqué par Bug #32147: Les variables dans Hobo ne sont pas supprimées quand on les supprime ajouté
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-deploy-notify-agents-on-object-deletion-7201.patch 0001-deploy-notify-agents-on-object-deletion-7201.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Pourquoi pre_delete et pas post_delete ? Je m'attendais à ce que pre_delete ne marche pas parce que la variable n'aurait pas encore été supprimée, au final les deux semblent fonctionner mais je le sens quand même mieux avec post_delete.
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud a écrit :
Pourquoi pre_delete et pas post_delete ? Je m'attendais à ce que pre_delete ne marche pas parce que la variable n'aurait pas encore été supprimée, au final les deux semblent fonctionner mais je le sens quand même mieux avec post_delete.
Effectivement ça me semble logique de le faire après le traitement (que ce soit save ou delete). Mais en fait ça n'a aucune importance, de toute façon le message est toujours envoyé à la fin de la requête (request_finished), dans post_environment_save on ne fait que dire "y'a une modif, il faudra l'envoyer dans rabbitmq à la fin du traitement de toute l'affaire". Bref, post_delete c'est plus logique, mais pre_delete marcherait tout aussi bien.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1e492e7aba803c2e7fd9f0b47f3812dee4e4cdec Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Oct 28 17:22:59 2020 +0100 deploy: notify agents on object deletion (#7201)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
deploy: notify agents on object deletion (#7201)