Development #41069
Flinguer les venv à chaque install
0%
Description
Ce serait plus propre.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
Je voudrais pas que ce soit le comportement par défaut mais avec une option pourquoi pas.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Emmanuel Cazenave a écrit :
Je voudrais pas que ce soit le comportement par défaut mais avec une option pourquoi pas.
Pourquoi ? On fait pas ça souvent et quand on le fera on oubliera l'option :/ Je préférerai que ce soit le comportement par défaut avec une option pour ne pas le faire.
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
Parce que relancer l'install est la méthode qu'on préconise pour mettre à jour son installation et que ça s'adresse aux gens qui sont pas trop au fait de la méthode manuelle (aller faire des git pull partout, migrate_schemas, etc) : https://doc-publik.entrouvert.com/dev/installation-developpeur/#Mettre-%C3%A0-jour-son-installation. Shooter les venv va beaucoup allonger le temps de cette procédure.
D'un autre coté j'ai fait relancé l'install des dizaines et des dizaines de fois, et évidement c'est pas infaillible, mais dans la très grande majorité des cas ça se passe bien tel que c'est actuellement.
J'ai pas suivi le problème de Paul qui a amené à la création de ce ticket, c'est pas une histoire liée au fait qu'il soit en unstable ? Je veux bien des détails en tous cas.
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
Et avec un option, si shooter le venv est le comportement que tu veux, tu la met dans ton inventaire local une bonne fois pour toute et tu es tranquille, voilà la tête du mien actuellement :
local: hosts: localhost: ansible_connection: local git_ssh: true compile_theme: false
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Donc on a deux stratégies, faire tout le temps un truc qui produits des erreurs bizarres mais est rapide, faire un truc un peu plus lent (franchement c'est rapide ça prend 5 minutes) mais qui évitera des erreurs que les gens ne comprennent pas. Je continue à penser que la stratégie amenant le moins d'erreur est à favoriser plutôt que ton confort personnel.
Mis à jour par Thomas Noël il y a environ 4 ans
Benjamin Dauvergne a écrit :
Donc on a deux stratégies, faire tout le temps un truc qui produits des erreurs bizarres mais est rapide, faire un truc un peu plus lent (franchement c'est rapide ça prend 5 minutes) mais qui évitera des erreurs que les gens ne comprennent pas. Je continue à penser que la stratégie amenant le moins d'erreur est à favoriser plutôt que ton confort personnel.
Moi j'aime bien ajouter des bidules dans le venv monté par devinst (ipython et tout ça) ça me ferait chier qu'il me flingue ça quand je mets à jour (et pas que je réinstalle, qui serait une option, why not).
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Frédéric Péters a écrit :
Mais c'était quoi l'erreur ?
kombu/celery qui ne se sont pas mis à jour vers les dernières versions compatible python3 (asynchronous->async) parce qu'on a pas de contrainte sur le setup.py de hobo.
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
D'accord setup.py pas à jour mais au lieu de créer un ticket là dessus, on crée un ticket dans devinst, et je dois lire des "confort personnels", wtf.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Emmanuel Cazenave a écrit :
D'accord setup.py pas à jour mais au lieu de créer un ticket là dessus, on crée un ticket dans devinst, et je dois lire des "confort personnels", wtf.
Si on s'accorde que publik-devinst c'est d'abord les développeurs d'EO et éventuellement pour d'autre, je ferme le ticket et je retourne dans ma caverne.
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
- Fichier 0001-add-clean_venv-option-41069.patch 0001-add-clean_venv-option-41069.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Emmanuel Cazenave
- Patch proposed changé de Non à Oui
Pour des besoin de debug ou pour les amateurs de ce mode de fonctionnement, mettre clean_venv: true
dans son inventaire locale.
Mis à jour par Paul Marillonnet il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Ok pour moi.
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
- Statut changé de Solution validée à Solution déployée
commit 149bfa569e406b4c2baf6d50bed97fb88cc3f3d3 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Thu Apr 2 08:52:23 2020 +0200 add clean_venv option (#41069)
add clean_venv option (#41069)