Repository navigation
fix(vector-store): fail remote connection tests instead of validating local fallbacks #59
Description
Finding
The connection-test API reports success for an unreachable remote Qdrant or Chroma endpoint because the backend constructor falls back to local disk storage.
Revision: 41ec093a745f1c0277893dc8f758b96b216c970f.
Reproduction
As an admin, POST /admin/api/vector-store/test with:
{
"provider": "qdrant",
"mode": "remote",
"url": "http://127.0.0.1:1",
"collection": "feasibility_unreachable"
}Repeat with provider=chroma. No service is listening on port 1.
Both requests returned HTTP 200, success=true, mode=remote. Their messages describe a different target:
Qdrant (embedded @ /app/data/qdrant_storage) is healthy
Chroma (persistent @ /app/data/chroma_storage) is healthy
This proves only that local fallback storage works, not that the requested remote service is reachable.
Relevant code
Expected outcome
- Connection tests must validate the exact requested target and fail when it cannot be reached.
- Explicit remote deployments must not silently change durability/storage location; any supported fallback should be opt-in and visibly degraded.
- Report configured and effective backend/mode separately where fallback is supported.
- Add unreachable/refused/timeout tests for both remote backends, plus successful live-remote checks.
Live remote Chroma passed a separate ingestion/restart test. Live remote Qdrant could not be exercised without an available approved server artifact/endpoint; this issue does not claim that its happy path failed.
Activity
spelech commentedon Oct 27, 2026
Triage & Implementation Brief
1. Scope & Effort
app/services/vector_store/manager.py,qdrant_store.py,chroma_store.py)2. Root Cause & Context
POST /admin/api/vector-store/testtests remote vector endpoints. However, store constructors catch remote connection errors and fall back to embedded local storage, causing the test endpoint to returnsuccess=true, mode=remotefor unreachable remote addresses (e.g.http://127.0.0.1:1).3. Implementation Plan
allow_fallback: bool = Trueparameter toQdrantStoreandChromaStoreconstructors./admin/api/vector-store/test), instantiate stores withallow_fallback=False.4. Acceptance Criteria
success=falsewith a descriptive connection failure.success=true.google-labs-jules commentedon Oct 27, 2026
Jules is on it. When finished, you will see another comment and be able to review a PR.
google-labs-jules commentedon Oct 27, 2026
Ready for a review! A PR has been created.