Skip to content

fix(ci): throttle Playwright shards and surface keycloak boot logs on failure - #2542

Open
shikanime wants to merge 3 commits into
mainfrom
fix/playwright-keycloak-flake
Open

fix(ci): throttle Playwright shards and surface keycloak boot logs on failure#2542
shikanime wants to merge 3 commits into
mainfrom
fix/playwright-keycloak-flake

Conversation

@shikanime

@shikanime shikanime commented Aug 21, 2026

Copy link
Copy Markdown
Member

Issues liées

#2541
#2543


Quel est le comportement actuel ?

Le run de merge-queue 32459083785 (PR #2537) échoue de façon intermittente sur les shards Playwright : les 4 shards (strategy.matrix.shard: [1,2,3,4], runs-on: runner-playwright) démarrent chacun leur propre stack complète, JVM Keycloak 26 incluse, en parallèle sur un même runner. Keycloak dépasse alors le budget du healthcheck (interval 5s × retries 10 = 50 s) et docker compose up avorte avec le conteneur dso-console_keycloak en Error. En cas d'échec, aucun log docker n'est remonté : le diagnostic est impossible depuis CI. Le healthcheck contient aussi une typo (fsS au lieu de -fsS).

Quel est le nouveau comportement ?

  • max-parallel: 2 sur la stratégie du job Playwright : 2 shards au plus bootent leurs stacks Keycloak simultanément, réduisant la contention CPU/RAM à l'origine du dépassement du healthcheck.
  • Bloc Dump docker compose state on failure (step if: failure()) : docker compose ps -a, logs --no-color keycloak, et docker inspect dso-console_keycloak (OOMKilled/ExitCode) — pour trancher OOM vs timeout directement depuis l'échec CI.
  • Correction de la typo du healthcheck fsS-fsS dans docker-compose.ci.yml (hygiène ; sans effet sur le flake, la typo étant inoffensive car curl renvoie le code de la dernière URL).

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Discussion liée : #2540. Run de référence : https://github.com/cloud-pi-native/console/actions/runs/32459083785

@shikanime shikanime added the CI/CD CI/CD stuffs label Aug 21, 2026
@shikanime shikanime self-assigned this Aug 21, 2026
@shikanime shikanime added this to the 9.24.5 milestone Aug 21, 2026
@github-actions github-actions Bot added the built label Aug 21, 2026
@shikanime
shikanime force-pushed the fix/playwright-keycloak-flake branch from e351539 to 613d7da Compare August 21, 2026 08:54
@shikanime
shikanime marked this pull request as ready for review August 21, 2026 08:57
iliesmrf
iliesmrf previously approved these changes Aug 21, 2026
@shikanime
shikanime added this pull request to the merge queue Aug 21, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 21, 2026
@shikanime
shikanime added this pull request to the merge queue Aug 21, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 21, 2026
@shikanime
shikanime force-pushed the fix/playwright-keycloak-flake branch from 613d7da to b6dbabf Compare August 21, 2026 12:29
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: Ifdc20eb6771981647bd97bb1c33028c06a6a6964
@shikanime
shikanime force-pushed the fix/playwright-keycloak-flake branch from 205f6c7 to 1a7e3e3 Compare August 21, 2026 13:31
@cloud-pi-native-sonarqube

Copy link
Copy Markdown

@shikanime
shikanime requested a review from KepoParis August 21, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

built CI/CD CI/CD stuffs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants