Skip to content

Fix qdrant/hdbscan healthchecks failing on missing curl/wget#49

Merged
elliottregan merged 1 commit intomainfrom
fix/healthchecks-missing-curl-wget
Apr 21, 2026
Merged

Fix qdrant/hdbscan healthchecks failing on missing curl/wget#49
elliottregan merged 1 commit intomainfrom
fix/healthchecks-missing-curl-wget

Conversation

@elliottregan
Copy link
Copy Markdown
Owner

Summary

  • qdrant/qdrant ships without curl and cspace-hdbscan-api ships without wget, so the healthchecks in docker-compose.core.yml fail every interval (/bin/sh: 1: curl: not found, 76-deep failing streak observed).
  • cspace-search-mcp declares depends_on: qdrant: service_healthy, so it never starts and cspace up hangs until compose's retry window expires, then fails with dependency failed to start: container <inst>.qdrant is unhealthy.
  • Swap to probes that use tools already present in each image: bash /dev/tcp for qdrant, python urllib.request for hdbscan (matches the already-healthy reduce-api probe one service above).

Test plan

  • Verified bash -c 'exec 3<>/dev/tcp/127.0.0.1/6333' succeeds inside a live qdrant/qdrant container.
  • Verified python -c 'import urllib.request; urllib.request.urlopen("http://localhost:8090/health")' returns 200 inside a live cspace-hdbscan-api container.
  • cspace up <planet> reaches [15/15] without a healthcheck-blocked search-mcp.

🤖 Generated with Claude Code

The qdrant/qdrant image ships without curl and cspace-hdbscan-api ships
without wget, so both healthchecks failed every interval. cspace-search-mcp
(depends_on qdrant: service_healthy) then never started, blocking
`cspace up` until the retry window expired. Swap qdrant to a bash /dev/tcp
probe and hdbscan to the python urllib probe already used by reduce-api.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 21, 2026

Deploy Preview for cspace-cli canceled.

Name Link
🔨 Latest commit 10a62db
🔍 Latest deploy log https://app.netlify.com/projects/cspace-cli/deploys/69e70191f76b3000089d90df

@elliottregan elliottregan merged commit 7de3c84 into main Apr 21, 2026
7 checks passed
@elliottregan elliottregan deleted the fix/healthchecks-missing-curl-wget branch April 21, 2026 04:49
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