Produits Entr'ouvert » Chrono

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

• URL : https://git.entrouvert.org/entrouvert/chrono/pulls/82
• Titre : WIP: Test trop long (#77590)
• Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/82/files

J'ai reproduit une bonne dizaine de fois en local, toujours ce même test qui prend 10 minutes.

Le test lancé en isolation prend un temps normal, ça n'aide pas.

Cette dizaine d'essai je l'ai passée à essayer de faire marcher un line profiler sur le test en question, quelle ligne de EventsFillslots.fillslot était à blâmer, mais c'est peine perdue pour trouver un outil qui fonctionne (pyinstrument s'en sort sur les tests simples mais produit un graphe vide sur le test en question).

J'attache mon meilleur résultat (un profiling du test lent, mais sans les lignes).

Et puis boum tout d'un coup je ne reproduis plus en local.

Je suis à peu près sûr que le patch de #76326 est coupable, c'est assez visible sur https://jenkins.entrouvert.org/job/chrono/buildTimeTrend que depuis on a des builds qui prennent du temps, en dents de scie.

Pas d'idées pour avancer davantage.

#76326 complète un test existant, en faisant une annulation de réservation:

        resp = app.post_json(
            '/api/agendas/recurring-events/fillslots/?action=update&agendas=%s' % agenda_slugs,
            params={
                'slots': events_to_book[1:],
                'user_external_id': 'user',
                'include_booked_events_detail': True,
                'check_overlaps': agenda_slugs,
            },
        )

En commentant

'include_booked_events_detail': True,

et en rajoutant dans le code un if pour faire le get_short_event_detail seulement si besoin:

            if payload.get('include_booked_events_detail'):
                cancelled_events = [
                    get_short_event_detail(
                        request,
                        events_by_id[x.event_id],
                        multiple_agendas=True,
                    )
                    for x in bookings_to_cancel
                ]

j'arrive à reproduire le test lent, ce n'est pas le code de #76326 qui ralentit les tests, mais bien l'annulation de 5 réservations sur 20 agendas contenant chaque un event récurrent sur 2 jours de la semaine.

Je reproduisais bien les requêtes lentes:
https://pad.entrouvert.org/chrono-requete-lente

Jusqu'à ce que j'essaie de faire un dump de ma base de test pour la passer à PierreD, pour qu'il regarde, que je fasse - --create-db (je suis en --reuse-db), et là, miracle, les tests redeviennent à peu près normaux.
J'ai quand même fourni à PierreD un dump et des logs pg pour analyse.

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

• URL : https://git.entrouvert.org/entrouvert/chrono/pulls/86
• Titre : WIP: api: don't buil details if not requested (#77590)
• Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/86/files

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

• URL : https://git.entrouvert.org/entrouvert/chrono/pulls/86

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

• URL : https://git.entrouvert.org/entrouvert/chrono/pulls/86
• Titre : tests, test_recurring_events_api_fillslots_multiple_agendas_queries est très long (#77590)
• Modifications : https://git.entrouvert.org/entrouvert/chrono/pulls/86/files