Projet

Général

Profil

Bug #63248

agent authentic : import anachronique aux signaux django_rbac

Ajouté par Paul Marillonnet il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
28 mars 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Qui casse notamment le script authentic2-multitenant.postinst


Demandes liées

Lié à Hobo - Development #63199: test_provision_user cassé après #57500 dans authenticFermé25 mars 2022

Actions

Révisions associées

Révision c9c76dea (diff)
Ajouté par Frédéric Péters il y a environ 2 ans

agent: wait for ready() signal to import from django_rbac (#63248)

Révision 3fd93457 (diff)
Ajouté par Frédéric Péters il y a environ 2 ans

Revert "agent: wait for ready() signal to import from django_rbac (#63248)"

This reverts commit c9c76dea77c2d3d453fd437f2b0f315dfcb5d849, it was
actually caused by authentic not yet being tagged.

Historique

#1

Mis à jour par Paul Marillonnet il y a environ 2 ans

Le fix serait quelque chose comme ça :

diff --git a/hobo/agent/authentic2/apps.py b/hobo/agent/authentic2/apps.py
index d628371..1905eaa 100644
--- a/hobo/agent/authentic2/apps.py
+++ b/hobo/agent/authentic2/apps.py
@@ -17,7 +17,6 @@
 from django.apps import AppConfig
 from django.conf import settings
 from django.db.models.signals import m2m_changed, post_save, pre_delete, pre_save
-from django_rbac.signals import post_soft_create, post_soft_delete

 class Plugin:
@@ -36,6 +35,8 @@ class Authentic2AgentConfig(AppConfig):
         return Plugin()

     def ready(self):
+        from django_rbac.signals import post_soft_create, post_soft_delete
+
         from . import provisionning

         engine = provisionning.Provisionning()

mais pas encore d’idée de comment tester.

#3

Mis à jour par Paul Marillonnet il y a environ 2 ans

  • Lié à Development #63199: test_provision_user cassé après #57500 dans authentic ajouté
#4

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Statut changé de Nouveau à Solution validée
  • Assigné à mis à Paul Marillonnet

mais pas encore d’idée de comment tester.

Ne bloquons pas là-dessus, tape ça en commit tag déploiement.

#5

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit c9c76dea77c2d3d453fd437f2b0f315dfcb5d849
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Mar 28 14:06:13 2022 +0200

    agent: wait for ready() signal to import from django_rbac (#63248)

(et je m'occupe tag etc.)

#6

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Fermé

+ revert parce qu'en fait c'est qu'il y a eu tag trop tôt sur hobo, que ça devait être déployé après un tag qui n'avait pas encore été fait sur authentic.

Formats disponibles : Atom PDF