Project

General

Profile

Development #5044

Besoin d'une commande pour liste les tenants

Added by Serghei Mihai almost 10 years ago. Updated over 9 years ago.

Status:
Fermé
Priority:
Normal
Start date:
24 June 2014
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Une commande "list_tenants" serait plus pratique que de passer par tenant_command pour liste les instances des tenants


Files

Associated revisions

Revision fbed23e8 (diff)
Added by Benjamin Dauvergne over 9 years ago

Add command list_tenants (fixes #5044)

refs #5106

History

#2

Updated by Frédéric Péters almost 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:
#3

Updated by Benjamin Dauvergne almost 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.

#4

Updated by Benjamin Dauvergne over 9 years ago

#5

Updated by Frédéric Péters over 9 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).

#6

Updated by Benjamin Dauvergne over 9 years ago

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.

#7

Updated by Benjamin Dauvergne over 9 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"...

#8

Updated by Frédéric Péters over 9 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).

#9

Updated by Benjamin Dauvergne over 9 years ago

  • File 0001-Add-command-list_tenants-fixes-5044.patch added

Ok.

#10

Updated by Benjamin Dauvergne over 9 years ago

  • File deleted (0001-Add-command-list_tenants-fixes-5044.patch)
#12

Updated by Thomas Noël over 9 years ago

Ok pour moi.

#13

Updated by Benjamin Dauvergne over 9 years ago

  • Status changed from Nouveau to Résolu (à déployer)
#14

Updated by Benjamin Dauvergne over 9 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF