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 2 months ago. Updated 9 days ago.

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

100%

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

0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch View (7.53 KB) Nicolas Roche, 13 Jun 2019 02:28 PM

0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch View (5.6 KB) Nicolas Roche, 21 Jun 2019 04:27 PM

0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch View (3.79 KB) Nicolas Roche, 21 Jun 2019 06:22 PM


Subtasks

Combo - Development #33948: rendre la commande import_site atomiqueSolution déployéeNicolas Roche

History

#1 Updated by Nicolas Roche 2 months 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 2 months 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 2 months 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 2 months 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.

#5 Updated by Nicolas Roche 2 months ago

oui, c'est plus simple comme ça.

#6 Updated by Nicolas Roche about 1 month ago

  • Status changed from Solution proposée to Rejeté

#7 Updated by Nicolas Roche about 1 month 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 about 1 month 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