fix: cleanup Qdrant container before and after integration tests #19

Merged
jcabillot merged 1 commits from fix/qdrant-cleanup into main 2026-06-22 12:04:48 -04:00
Owner

Problème

Le container Qdrant lancé pour les tests d'intégration n'est jamais nettoyé après l'exécution du workflow, que la run réussisse ou échoue. Sur un runner partagé, le container qdrant d'une run précédente persiste et bloque le démarrage du suivant :

docker: Error response from daemon: Conflict. The container name "/qdrant" is already in use...

Voir run #1958 job #4795.

Solution

Deux mesures de protection :

  1. docker rm -f qdrant avant docker run — tue le container s'il existe encore avant d'en lancer un nouveau
  2. Step Cleanup Qdrant avec if: always() — nettoie le container après les tests, que les tests passent ou échouent

Les deux étapes utilisent 2>/dev/null || true pour ne jamais casser la pipeline si le container n'existe pas.

## Problème Le container Qdrant lancé pour les tests d'intégration n'est jamais nettoyé après l'exécution du workflow, que la run réussisse ou échoue. Sur un runner partagé, le container `qdrant` d'une run précédente persiste et bloque le démarrage du suivant : ``` docker: Error response from daemon: Conflict. The container name "/qdrant" is already in use... ``` Voir run #1958 job #4795. ## Solution Deux mesures de protection : 1. **`docker rm -f qdrant` avant `docker run`** — tue le container s'il existe encore avant d'en lancer un nouveau 2. **Step `Cleanup Qdrant` avec `if: always()`** — nettoie le container après les tests, que les tests passent ou échouent Les deux étapes utilisent `2>/dev/null || true` pour ne jamais casser la pipeline si le container n'existe pas.
opencodecabilloteu added 1 commit 2026-06-22 11:54:15 -04:00
fix: cleanup Qdrant container before and after integration tests
Docker Build and Push / test (pull_request) Successful in 20s
Docker Build and Push / integration-test (pull_request) Successful in 26s
Docker Build and Push / build (pull_request) Successful in 1m29s
c8e4d68238
jcabillot merged commit 5158b0b093 into main 2026-06-22 12:04:48 -04:00
jcabillot deleted branch fix/qdrant-cleanup 2026-06-22 12:04:49 -04:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: perso/mcp-maildir#19