From 722f639fa56c169f8f4081085079e081cad5dec8 Mon Sep 17 00:00:00 2001 From: BukeLy Date: Thu, 20 Nov 2025 00:26:36 +0800 Subject: [PATCH] fix: remove Qdrant health check in E2E workflow Why this change is needed: Qdrant Docker image does not have curl or wget pre-installed, causing health check to always fail and container to be marked as unhealthy after timeout. How it solves it: Remove health check from Qdrant service container configuration. The E2E test already has a "Wait for Qdrant" step that uses curl from the runner environment to verify service readiness before running tests. Impact: - Qdrant container will start immediately without health check delays - Service readiness still verified by test-level wait step - Eliminates container startup failures Testing: Next CI run should successfully start Qdrant container and pass the wait/verify steps in the test workflow. --- .github/workflows/e2e-tests.yml | 5 ----- 1 file changed, 5 deletions(-) diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index dea2d1d3..a48edec7 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -98,11 +98,6 @@ jobs: ports: - 6333:6333 - 6334:6334 - options: >- - --health-cmd "wget --no-verbose --tries=1 --spider http://localhost:6333/health || exit 1" - --health-interval 10s - --health-timeout 5s - --health-retries 5 strategy: matrix: