Development #70164
Tox : ne pas utiliser xdist par défaut
0%
Description
Plutôt faire en sorte que Jenkins envoie les options xdist qui vont bien.
La motivation est qu'en local on ne peut pas utiliser le bien pratique --pdb, sauf à ajouter un -n 0 --dist no, dont il faut se rappeler, puis oublier, puis se le noter quelque part etc.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a plus d'un an
De #70155#note-13 :
Agate Berriot a écrit :
J'avais prévu une variable d'environnement
NUMPROCESSES(à 1 par défaut dans le tox.ini) pour ça, visiblement ça ne fonctionne pas ?
Ça fonctionne au sens où il y a bien un seul processus d'utilisé en local, mais ça ne fonctionne pas au sens où même avec un seul processus le plugin xdist reste dans la partie et si tu passes l'option --pdb :
ERROR: --pdb is incompatible with distributing tests; try using -n0 or -nauto.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Assigné à changé de Emmanuel Cazenave à Benjamin Dauvergne
Mis à jour par A. Berriot il y a plus d'un an
Ah oui d'accord ! On pourrait simplement le passer à 0 par défaut?
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-tox.ini-run-py.test-without-any-option-when-using-po.patch 0001-tox.ini-run-py.test-without-any-option-when-using-po.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le souci vient de "--dist loadfile", sans ça et avec -n0 ou -nauto ça passe. Comme je me dis que ce "--dist loadfile" est là certainemnt pour une bonne raison, le plus simple c'est de laisser la ligne d'argument vide si on utilise tox avec des options pour pytest.
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne a écrit :
Le souci vient de "--dist loadfile", sans ça et avec -n0 ou -nauto ça passe. Comme je me dis que ce "--dist loadfile" est là certainemnt pour une bonne raison, le plus simple c'est de laisser la ligne d'argument vide si on utilise tox avec des options pour pytest.
ouais, le dist=loadfile dit à xdist de grouper et distribuer les tests par fichiers, plutôt que de faire une distribution aléatoire dans les processus. Sans ça, ça casse dans wcs et c'est fort triste, d'ailleurs.
(je valide au passage, le patch me semble très bien)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 5c250891aaa1e577a30b06de819452b2973db926
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Wed Oct 12 14:13:33 2022 +0200
tox.ini: run py.test without any option when using posargs (#70164)Mis à jour par Frédéric Péters il y a plus d'un an
Point cycle de mise à jour : même ces commits innocents m'ennuient, parce qu'ils font figurer le module dans la page que j'utilise pour suivre les modules qui devraient être taggués (https://git.entrouvert.org/lag.html), ça oblige du coup à aller vérifier et se dire que ok celui-ci est insignifiant.
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
tox.ini: run py.test without any option when using posargs (#70164)