Développement #8217
Ajouter une tâche hobo pour notifier les agents des nouveaux rôles, connecter cette tâche aux signaux post_save, post_delete sur les rôles
100%
Description
Pour permettre aux agents de savoir pour quel tenant sont destinés ces messages de provisionning un champ audience est ajouté, contenant la listes des entity_id SAML connus d'authentic.
Files
Related issues
Associated revisions
add new agent task to provision objects to tenants (fixes #8217)
First use is to connect it to post_save, post_delete signal on Role
model of authentic, to propagate roles to tenants.
update french translations (#8217)
agent/authentic2: do not activate new and old role import at the same time (#8217)
agent/authentic2: fix typos in import-wcs-roles (#8217)
History
Updated by Benjamin Dauvergne about 10 years ago
- Related to Gestion de projet #8006: Supprimer la création de rôles dans w.c.s. added
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
- Patch proposed changed from No to Yes
Updated by Benjamin Dauvergne about 10 years ago
- Related to Développement #8219: nouvelle command 'notify' pour traiter les notifications provisionnement/déprovisionnement des rôles en provenance d'authentic added
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
Nouvelle version où on envoie tous les rôles à chaque fois, ça mange pas de pain et ça simplifie la synchro.
Updated by Frédéric Péters about 10 years ago
Je préférerais une version qui utilise les classes en place dans services.py, un notify() qui soit à peu près
for klass in service_classes:
klass.notify(data)
Updated by Benjamin Dauvergne about 10 years ago
- File deleted (
0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch)
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
Pas le bon patch. Ok pour le for klass.
Updated by Frédéric Péters about 10 years ago
Mmmm, j'imaginais le notify() dans la classe de base, utilisant self.service_manage_cmd.
if not os.path.exists(settings.WCS_MANAGE_COMMAND):
Et je note un autre truc là-dessus maintenant, c'est que le WCS_MANAGE_COMMAND, ce n'est pas nécessairement un fichier (en local j'ai 'wcsctl.py -f fichier.cfg' pour wcs et '.../venv/python ..../manage.py' pour d'autres applications).
Je n'ai pas d'autre suggestion que de faire le subprocess.Popen dans toutes les situations et ignorer le OSError qu'il lèverait en cas de commande manquante.
Updated by Benjamin Dauvergne about 10 years ago
Je savais bien que je n'avais pas envie d'aller vers ça :)
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-new-notify-command-to-handle-role-provisionning-depr.patch added
Updated by Benjamin Dauvergne about 10 years ago
- File deleted (
0001-new-notify-command-to-handle-role-provisionning-depr.patch)
Updated by Frédéric Péters about 10 years ago
Avec la suppression de la modif au settings.py, ça me semble bien pour être commité.
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
Voilà sur le bon ticket (je me suis mis à parler de ce ticket sur l'autre aussi...).
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch 0001-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
Voilà avec la suppression de la modifications à settings.py.
Updated by Frédéric Péters about 10 years ago
Après avoir dormi dessus, je préférerais qu'on appelle la commande hobo_notify.
À part ça, je me suis aussi dit que j'allais devoir réfléchir à comment faire fonctionner ça dans l'environnement imio où il n'y a pas pour le moment pas de hobo, mais c'est mon affaire.
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-import-wcs-roles-import-roles-.emails-and-.emails_to.patch 0001-import-wcs-roles-import-roles-.emails-and-.emails_to.patch added
- File 0002-add-new-agent-task-to-provision-objects-to-tenants-f.patch 0002-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
- File 0003-agent-authentic2-do-not-activate-new-and-old-role-im.patch 0003-agent-authentic2-do-not-activate-new-and-old-role-im.patch added
- modification à import-wcs-roles pour créer un existant au niveau des champs emails et emails_to_members
- plein de fichiers manquants: la surcharge du formulaire authentic des rôles (pour la gestion des emails de rôle), mise à jour du fichier sudoers dans le paquet (pour pouvoir lancer les commands hobo_notify des différentes applications), une implémentation de hobo_notify commune (qui ne fait rien, qui pourra très vite propager les rôles à Combo)
- un setting HOBO_ROLE_EXPORT pour contrôler qui de import-wcs-roles ou de hobo_notify importe les rôles, pour gérer la transition
- le coup du OSError ne peut pas marcher parce qu'on utilise sudo (et sudo est toujours là contrairement à la commande hobo_notify), donc ça va systématiquement faire un returncode!=0 au lieur d'un OSError, que je log mais c'est pas bien, il faudrait trouver un moyen de détecter qu'un service est là ou pas.
Voilà tout ça est déployé sur montpellier.dev.eo (avec HOBO_ROLE_EXPORT=True) mes quelques tests indiquent que ça marche (création, suppression, modification des champs).
Updated by Benjamin Dauvergne about 10 years ago
- Blocked by Développement #8254: dans /api/roles exporter aussi .emails et .emails_to_members added
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-import-wcs-roles-import-roles-.emails-and-.emails_to.patch 0001-import-wcs-roles-import-roles-.emails-and-.emails_to.patch added
- File 0002-add-new-agent-task-to-provision-objects-to-tenants-f.patch 0002-add-new-agent-task-to-provision-objects-to-tenants-f.patch added
- File 0003-agent-authentic2-do-not-activate-new-and-old-role-im.patch 0003-agent-authentic2-do-not-activate-new-and-old-role-im.patch added
- importation de role['details'] dans Role.description
- ajout des fichiers .po .mo du package agent/authentic2 dans le MANIFEST.in
Updated by Frédéric Péters about 10 years ago
Pour les traductions, je préfère qu'on garde un unique .po, ça t'irait ?
Updated by Frédéric Péters about 10 years ago
Il n'y a plus besoin du recursive-include hobo/agent/authentic2/locale *.po *.mo dans le MANIFEST.in.
Tu pourrais ajouter les lignes de commentaire de la GPL en début des nouveaux fichiers .py ? (il y a l'exemple dans hobo/profile/*.py).
Je préfère la modification du .po dans un commit séparé, je trouve que ça facilite les manipulations de commits.
En prenant ça en compte, c'est bon pour moi sans relecture supplémentaire.
Updated by Benjamin Dauvergne about 10 years ago
- Status changed from En cours to Résolu (à déployer)
- % Done changed from 0 to 100
Appliqué par commit 57b1bd5c41c5f9d55796df351cd9101e1bba25a9.
Updated by Benjamin Dauvergne about 10 years ago
- Status changed from Résolu (à déployer) to Fermé
import-wcs-roles: import roles .emails and .emails_to_members fields (#8217)
As we are moving from and w.c.s. role master to an authentic as role
master, we must first import as much data from w.c.s. roles as possible
into authentic.