Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(PC-34429)[API] ci: use pytest-split to parallelize tests in GHA #16217

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fseguin-pass
Copy link
Contributor

@fseguin-pass fseguin-pass commented Feb 6, 2025

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-34429

Afin d'accélérer le temps d'exécution de la CI, on améliore le découpage et la répartition des tests unitaires du backend grâce à la lib pytest-splits.
L'algorithme de répartition utilise un fichier mis en cache Github (pour chaque SHA de commit) qui stocke les timings d'exécution par les runners.
On exécute alors une matrice de 8 jobs:

  • 4 jobs pour les 3020 tests des routes du backoffice
  • 4 jobs pour les 7847 autres tests

@fseguin-pass fseguin-pass requested review from a team as code owners February 6, 2025 11:21
@fseguin-pass fseguin-pass force-pushed the fseguin/PC-34429-better-pytest-splitting branch 8 times, most recently from 8f8c074 to f00bf3e Compare February 7, 2025 12:57
@pass-culture pass-culture deleted a comment from github-actions bot Feb 7, 2025
Copy link
Contributor

github-actions bot commented Feb 7, 2025

Visit the preview URL for this PR (updated for commit 70228fd):

https://pc-pro-testing--pr16217-fseguin-pc-34429-bet-azz1cbgc.web.app

(expires Sun, 09 Feb 2025 16:41:57 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1

@fseguin-pass fseguin-pass force-pushed the fseguin/PC-34429-better-pytest-splitting branch 5 times, most recently from 70228fd to d2a4cbc Compare February 7, 2025 16:51
@fseguin-pass fseguin-pass force-pushed the fseguin/PC-34429-better-pytest-splitting branch from d2a4cbc to 2e81728 Compare February 11, 2025 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant