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
Fichiers
Sous-tâches
Historique
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ce patch ajoute l'option "--if-empty"
à la commande "import_template
" lancée via "hobo_deploy
".
Mis à jour par Frédéric Péters il y a presque 5 ans
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).
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch ajouté
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.
Mis à jour par Frédéric Péters il y a presque 5 ans
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.
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch 0003-hobo_deploy-do-not-recall-import_template-when-it-pr.patch ajouté
oui, c'est plus simple comme ça.
Mis à jour par Nicolas Roche il y a presque 5 ans
- Statut changé de Rejeté à Solution proposée
J'ai rejeté le ticket par erreur (du fait que j'ai rejeté le ticket chapeau).
Mis à jour par Frédéric Péters il y a presque 5 ans
- Sujet changé de rendre "import_template" atomique afin qu'elle ne puisse plus être rejouée dès lors qu'elle à abouti une première fois à rendre "import_template" atomique afin qu'elle ne puisse plus être rejouée dès lors qu'elleaà abouti une première fois
Mis à jour par Frédéric Péters il y a 10 mois
- Statut changé de Solution proposée à Fermé
(cette histoire import-template se remplace/ra par des applications).