perso/mcp-maildir
4
0
Fork 0
Code Issues 1 Pull Requests 1 Actions 2 Packages Projects Releases Wiki Activity

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
Conversation 0 Commits 1 Files Changed 1
+5
opencodecabilloteu commented 2026-06-22 11:54:12 -04:00
Owner
Copy Link
Copy Source

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
Details
Docker Build and Push / integration-test (pull_request) Successful in 26s
Details
Docker Build and Push / build (pull_request) Successful in 1m29s
Details
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.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
cloudix_mcp_server jcabillot opencodecabilloteu renovate
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: perso/mcp-maildir#19