Development #67550
nightly build cassé à cause du test_massive_role_parenting
0%
Description
Dans le job principal, certains builds sont déclenchés lorsqu’un nouveau commit arrive sur la branche main. D’autres en revanche sont déclenchés de façon périodique, chaque nuit, et sont censés exécuter une version plus extensive des tests. C’est ce build nocturne qui est cassé, car contrairement aux autres il exécute les tests_rbac/test_rbac.py.
L’un des tests, test_massive_role_parenting, sur le temps de calcul des permissions dans le cas de rôle avec relations d’héritage à rallonge, s’exécute plus lentement que la limite qu’on avait fixée arbitrairement, et provoque le crash du build.
Fichiers
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a presque 2 ans
En jouant avec le nombre de processes pytest (via pytest-xdist) j’arrive à reproduire en local. On avait déjà, dans un ticket précédent que je ne retrouve plus, évoqué le fait que le temps d’exécution peut varier et peut difficilement être choisi comme critère de réussite d’un test donné.
Est-ce qu’on ne dégagerait toutes les parties de ce test qui vérifient que l’exécution ne prend pas plus qu’un certain temps donné ?
Mis à jour par Valentin Deniaud il y a presque 2 ans
Paul Marillonnet a écrit :
Est-ce qu’on ne dégagerait toutes les parties de ce test qui vérifient que l’exécution ne prend pas plus qu’un certain temps donné ?
Go go go
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Fichier 0001-tests_rbac-remove-arbitrary-execution-time-checks-67.patch 0001-tests_rbac-remove-arbitrary-execution-time-checks-67.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Valentin Deniaud a écrit :
Paul Marillonnet a écrit :
Est-ce qu’on ne dégagerait toutes les parties de ce test qui vérifient que l’exécution ne prend pas plus qu’un certain temps donné ?
Go go go
Avec les tests_rbac ajouté dans tox.ini pour qu’ils soient pris par le build du job wip. Je vais tourner plusieurs fois le build pour voir si j’arrive à planter les tests malgré le retrait de ces vérifications sur le temps d’exécution.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Moi du moment que jenkins est content je suis content
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4263b860a1bd376ebfcf3c583cb8107dba312f0e (HEAD -> main, ssh/main)
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Wed Jul 20 16:01:11 2022 +0200
tests_rbac: remove arbitrary execution time checks (#67550)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests_rbac: remove arbitrary execution time checks (#67550)