Development #5044
Besoin d'une commande pour liste les tenants
100%
Description
Une commande "list_tenants" serait plus pratique que de passer par tenant_command pour liste les instances des tenants
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a presque 10 ans
- Fichier 0002-management-command-for-listing-tenants.patch 0002-management-command-for-listing-tenants.patch ajouté
Patch joint
Mis à jour par Frédéric Péters il y a plus de 9 ans
- Patch proposed mis à Non
Avec les tenants sans modèle (#5106), ce patch ne s'applique plus (enfin, il s'applique sans doute mais le résultat est mauvais); à lire le code de "0009-Add-command-to-create-missing-tenant-schemas.patch", le taf pourrait être faire par ces lignes :
connection.set_schema_to_public()
all_tenants = TenantMiddleware.get_tenants()
for tenant in all_tenants:
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Assigné à changé de Serghei Mihai à Benjamin Dauvergne
Si ça ne dérange personne je vais m'assigner ce ticket pour le traiter avec #5106.
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Fichier 0001-Add-command-list_tenants-fixes-5044.patch 0001-Add-command-list_tenants-fixes-5044.patch ajouté
- % réalisé changé de 0 à 100
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 9 ans
Ça m'irait bien que la commande réussisse quand même quand il n'y a aucun tenant (sans objection à l'affichage d'un message sur stderr dans cette situation).
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Fichier 0001-Add-command-list_tenants-fixes-5044.patch 0001-Add-command-list_tenants-fixes-5044.patch ajouté
Si il n'y a pas de tenants je fais juste un sys.exit(1) et j'ai enlevé le caractère : entre le schéma et le nom de domaine pour que soit un plus facile à parser avec les outils classiques.
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
C'est là que je me rend compte que je n'ai peut-être pas bien répondu au "réussisse quand même"...
Mis à jour par Frédéric Péters il y a plus de 9 ans
Le sys.exit(1) ça marque aussi une erreur; pour moi on peut juste dérouler une liste vide et terminer tranquille. (le cas auquel je pense, c'est l'agent de déploiement, qui peut vouloir faire un list_tenants pour voir si c'est un nouveau site ou pas, et pour ça, je préfère une commande qui ne retourne rien, avec succès, qu'une commande qui va parfois sortir avec un code d'erreur dans une situation où il n'y a pas d'erreur).
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Fichier 0001-Add-command-list_tenants-fixes-5044.patch ajouté
Ok.
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Fichier
0001-Add-command-list_tenants-fixes-5044.patchsupprimé
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Fichier 0001-Add-command-list_tenants-fixes-5044.patch 0001-Add-command-list_tenants-fixes-5044.patch ajouté
Erreur, avec le bon patch.
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
Appliqué par commit fbed23e8e3ef59f52372e063f5d361b11f91fd2b.
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Statut changé de Résolu (à déployer) à Fermé
Add command list_tenants (fixes #5044)
refs #5106