Project

General

Profile

Development #41069

Flinguer les venv à chaque install

Added by Benjamin Dauvergne 4 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
26 Mar 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Ce serait plus propre.

0001-add-clean_venv-option-41069.patch View (949 Bytes) Emmanuel Cazenave, 02 Apr 2020 08:52 AM

Associated revisions

Revision 149bfa56 (diff)
Added by Emmanuel Cazenave 4 months ago

add clean_venv option (#41069)

History

#1 Updated by Emmanuel Cazenave 4 months ago

Je voudrais pas que ce soit le comportement par défaut mais avec une option pourquoi pas.

#2 Updated by Benjamin Dauvergne 4 months ago

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.

#3 Updated by Emmanuel Cazenave 4 months ago

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.

#4 Updated by Emmanuel Cazenave 4 months ago

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

#5 Updated by Benjamin Dauvergne 4 months ago

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.

#6 Updated by Frédéric Péters 4 months ago

Mais c'était quoi l'erreur ?

#7 Updated by Thomas Noël 4 months ago

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).

#8 Updated by Benjamin Dauvergne 4 months ago

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.

#9 Updated by Emmanuel Cazenave 4 months ago

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.

#10 Updated by Benjamin Dauvergne 4 months ago

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.

#11 Updated by Emmanuel Cazenave 4 months ago

Pour des besoin de debug ou pour les amateurs de ce mode de fonctionnement, mettre clean_venv: true dans son inventaire locale.

#12 Updated by Paul Marillonnet 4 months ago

  • Status changed from Solution proposée to Solution validée

Ok pour moi.

#13 Updated by Emmanuel Cazenave 4 months ago

  • Status changed from Solution validée to 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)

Also available in: Atom PDF