Project

General

Profile

Développement #86737

test test_studio_home_recent_changes qui parfois échoue

Added by Valentin Deniaud 10 months ago. Updated 10 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
08 February 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Pas réussi à reproduire sur main, mais sur une de mes branches qui ajoute suffisamment de tests pour bousculer l'ordre xdist, badaboum https://jenkins.entrouvert.org/job/gitea/job/wcs/job/wip%252F83593-testdef-socle-de-base-pour-les-w/2/

Même veine que #86736, les objets stockés via SQL ne sont pas isolés entre les workers contrairement aux objets stockés en XML.

Associated revisions

Revision 9101bd1e (diff)
Added by Valentin Deniaud 10 months ago

tests: make snapshot count stable in test_studio_home_recent_changes (#86737)

History

#1

Updated by Robot Gitea 10 months ago

  • Status changed from Nouveau to En cours

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#2

Updated by Valentin Deniaud 10 months ago

  • Description updated (diff)
#3

Updated by Robot Gitea 10 months ago

  • Status changed from En cours to Solution proposée
#4

Updated by Frédéric Péters 10 months ago

SQL ne sont pas isolés entre les workers

Pour être précis, entre workers, c'est isolé.

    prefix = os.environ.get('PYTEST_XDIST_WORKER')
    ...
        dbname = 'wcstests_%s_%d' % (prefix, random.randint(0, 100000))

Et le truc ici doit être qu'il y a décalage dans l'enchainement des modules sur lesquels joue le même worker. (ça ne change rien au patch qui est correct).

#5

Updated by Robot Gitea 10 months ago

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

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#6

Updated by Robot Gitea 10 months ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#7

Updated by Transition automatique 10 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#8

Updated by Transition automatique 8 months ago

Automatic expiration

Also available in: Atom PDF