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
Files
Associated revisions
History
Updated by Serghei Mihai over 10 years ago
- File 0002-management-command-for-listing-tenants.patch 0002-management-command-for-listing-tenants.patch added
Patch joint
Updated by Frédéric Péters about 10 years ago
- Patch proposed set to No
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:
Updated by Benjamin Dauvergne about 10 years ago
- Assignee changed from Serghei Mihai to Benjamin Dauvergne
Si ça ne dérange personne je vais m'assigner ce ticket pour le traiter avec #5106.
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-Add-command-list_tenants-fixes-5044.patch 0001-Add-command-list_tenants-fixes-5044.patch added
- % Done changed from 0 to 100
- Patch proposed changed from No to Yes
Updated by Frédéric Péters about 10 years ago
Ç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).
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-Add-command-list_tenants-fixes-5044.patch 0001-Add-command-list_tenants-fixes-5044.patch added
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.
Updated by Benjamin Dauvergne about 10 years ago
C'est là que je me rend compte que je n'ai peut-être pas bien répondu au "réussisse quand même"...
Updated by Frédéric Péters about 10 years ago
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).
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-Add-command-list_tenants-fixes-5044.patch added
Ok.
Updated by Benjamin Dauvergne about 10 years ago
- File deleted (
0001-Add-command-list_tenants-fixes-5044.patch)
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-Add-command-list_tenants-fixes-5044.patch 0001-Add-command-list_tenants-fixes-5044.patch added
Erreur, avec le bon patch.
Updated by Benjamin Dauvergne about 10 years ago
- Status changed from Nouveau to Résolu (à déployer)
Appliqué par commit fbed23e8e3ef59f52372e063f5d361b11f91fd2b.
Updated by Benjamin Dauvergne about 10 years ago
- Status changed from Résolu (à déployer) to Fermé
Add command list_tenants (fixes #5044)
refs #5106