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
100%
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
Subtasks
History
#1 Updated by Nicolas Roche 8 months ago
- File 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch View added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Ce patch ajoute l'option "--if-empty" à la commande "import_template" lancée via "hobo_deploy".
#2 Updated by Frédéric Péters 7 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 7 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 7 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 7 months ago
oui, c'est plus simple comme ça.
#6 Updated by Nicolas Roche 7 months ago
- Status changed from Solution proposée to Rejeté
#7 Updated by Nicolas Roche 7 months 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 7 months 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