Project

General

Profile

Development #33875

rendre "import_template" atomique afin qu'elle ne puisse plus être rejouée dès lors qu'elleaà abouti une première fois

Added by Nicolas Roche over 3 years ago. Updated over 3 years ago.

Status:
Solution proposée
Priority:
Normal
Assignee:
Category:
agent
Target version:
-
Start date:
13 June 2019
Due date:
% Done:

100%

Estimated time:
(Total: 0:00 h)
Patch proposed:
Yes
Planning:
No

Description

Il s'agit de faire en sorte que la commande cook reste idempotente/résiliente (© thomas),
et donc que la commande "import_template" qu'elle lance n'écrase pas les éventuelles modifications apportées entre temps par l'usager.

C'est le comportement par défaut observé sur wcs :
wcs/ctl/check_hobos.py::CmdCheckHobos:deploy()

 if not os.path.exists(pub.app_dir):

Pour les autres briques, la commande "import_site" doit être appelée avec --if-empty


Files


Subtasks

Combo - Development #33948: rendre la commande import_site atomiqueFerméNicolas Roche

Actions

History

#1

Updated by Nicolas Roche over 3 years ago

Ce patch ajoute l'option "--if-empty" à la commande "import_template" lancée via "hobo_deploy".

#2

Updated by Frédéric Péters over 3 years ago

        parser.add_argument('--if-empty', action='store_true', default=False,
                            help='Import only if passerelle is empty')

Je dirais que par défaut ça doit être True (+ corriger le texte d'aide).

#3

Updated by Nicolas Roche over 3 years ago

J'ai remplacé '--if-empty' par une option '--again' car '--overwrite' est déjà utilisée dans l'import de passerelle.
Du coup '--force' perd son sens car on ne saurait pas ce que l'on force.
Peut-être que '--re-import' serait plus joli, dites-moi.

#4

Updated by Frédéric Péters over 3 years ago

Là j'en suis à me dire que non non, que ça soit viré, pas d'option, un seul comportement. Et laisser les variations aux commandes natives (import_site). (parce que vraiment aucun sens de dupliquer l'import_site sous un autre nom et avec d'autres noms d'options).

Ou bien il y a un sens que je ne pige pas.

#6

Updated by Nicolas Roche over 3 years ago

  • Status changed from Solution proposée to Rejeté
#7

Updated by Nicolas Roche over 3 years ago

  • Status changed from Rejeté to Solution proposée

J'ai rejeté le ticket par erreur (du fait que j'ai rejeté le ticket chapeau).

#8

Updated by Frédéric Péters over 3 years ago

  • Subject changed from rendre "import_template" atomique afin qu'elle ne puisse plus être rejouée dès lors qu'elle à abouti une première fois to rendre "import_template" atomique afin qu'elle ne puisse plus être rejouée dès lors qu'elleaà abouti une première fois

Also available in: Atom PDF