Skip to content

docs: fix drift in Docker Deployment - #308

Draft
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-deployment-docker-20260817
Draft

docs: fix drift in Docker Deployment#308
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-deployment-docker-20260817

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

First audit of this page. Supersedes stale #257 (opened 2026-07-15, never merged).

Five code-anchored claims corrected against automem@42ba8b6.

Claim Current state Fix Evidence
/health returns "qdrant": "not_configured" when Qdrant is unavailable health_data always sets "qdrant": "connected" if qdrant_available else "disconnected""not_configured" appears nowhere in the service Changed value to "disconnected" health.py#L90
FalkorDB healthcheck test command is redis-cli ping The compose healthcheck is password-aware: redis-cli -a "$REDIS_PASSWORD" ping when a password is set, else redis-cli ping Documented both branches docker-compose.yml#L21
FLASK_DEBUG=1 enables debug mode + hot-reload (edit a file → auto-reload in ~2s) The app always calls app.run(host="::", port=port, debug=False) — the reloader is off, so FLASK_DEBUG/FLASK_ENV have no effect and there is no hot-reload; the bind mount only makes source visible, a restart is required Rewrote the "Debug Mode" row and the hot-reload section to describe the restart-to-apply workflow runtime_wiring.py#L109
Production recommendation: set LOG_LEVEL=INFO or WARNING LOG_LEVEL is not read anywhere in the service (repo-wide grep is empty) Removed the fabricated variable from the recommendation grep LOG_LEVEL over automem@42ba8b6 → no hits
Mermaid labels the API service memory-service The Docker Compose service key is flask-api (the Service Details table already uses it); memory-service is the Railway name Renamed the mermaid node to flask-api docker-compose.yml#L37

Verified against: automem@42ba8b61b7d0b24ecaeb7feb4ceef59f09fc7cd0

Questions

None.

Unverified

None — every corrected claim was located in current source.

Follow-ups

  • The host-side port columns in the Port Mappings and Service Details tables show fixed ports, but compose makes them overridable (${FALKORDB_HOST_PORT:-6379}, etc.). Defaults are correct, so this is a simplification rather than drift; left as-is.

Draft because two fixes rewrite explanatory prose (the hot-reload section and the Debug Mode row).


Generated by Claude Code

Correct five code-anchored claims against automem@42ba8b6:
- /health reports "qdrant": "disconnected" (not "not_configured") when Qdrant is down (automem/api/health.py L90)
- FalkorDB healthcheck is password-aware, not a bare redis-cli ping (docker-compose.yml L21)
- app.run(debug=False) is hardcoded, so FLASK_DEBUG=1 does not enable the reloader; there is no hot-reload (automem/runtime_wiring.py L109)
- LOG_LEVEL is not read anywhere in the service; drop it from the production recommendation
- Docker compose service is flask-api, not memory-service (docker-compose.yml L37)

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015PuVebr9vp2BdLnQfXo3AC
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying automem-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: c9485d8
Status: ✅  Deploy successful!
Preview URL: https://c8eb6fc4.automem-website.pages.dev
Branch Preview URL: https://docs-audit-deployment-docker-57bm.automem-website.pages.dev

View logs

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.

2 participants