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
Files
Subtasks
History
Updated by Nicolas Roche over 3 years ago
- File 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 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
".
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).
Updated by Nicolas Roche over 3 years ago
- File 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch added
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.
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.
Updated by Nicolas Roche over 3 years ago
- File 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch added
oui, c'est plus simple comme ça.
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).
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