Skip to content

docs: capture deployment gotchas (TLS defaults, app-key match, local embeddings)#98

Merged
vineethkrishnan merged 1 commit into
mainfrom
docs/deploy-learnings
Jun 14, 2026
Merged

docs: capture deployment gotchas (TLS defaults, app-key match, local embeddings)#98
vineethkrishnan merged 1 commit into
mainfrom
docs/deploy-learnings

Conversation

@vineethkrishnan

Copy link
Copy Markdown
Owner

What

Documents the real issues encountered during a self-hosted deployment, so the next person doesn't rediscover them.

Changes

  • docker-deployment.md: migrations are now auto-run by the entrypoint (the old "manual migrations" text was stale); added a TLS section for DATABASE_SSL / REDIS_TLS with the bundled plain Postgres/Redis.
  • reference/configuration.md: added DATABASE_SSL, REDIS_TLS, EMBEDDING_CACHE_DIR, the agent LLM provider, local-embedding defaults; clarified GITHUB_PRIVATE_KEY is key contents from the same App that's installed.
  • llm-providers.md: new Embeddings (PR memory) section (voyage vs local, dimensions, glibc-image requirement).
  • faq.md + setup-walkthrough.md: troubleshooting for
    • crash-loop from production TLS defaults (The server does not support SSL connections / Redis timeout),
    • 404 on installation token from mismatched GITHUB_APP_ID/key,
    • empty installations/repositories when installed before creds were correct,
    • "Indexed 0 comments" being expected on repos with no review history.

Notes

All generic, no environment-specific details. npm run docs:build passes. Touches docs-site/**, so the docs-deploy workflow will publish on merge.

…embeddings)

Fold the issues hit during a real self-hosted bring-up into the docs and troubleshooting.

- docker-deployment: correct the stale "migrations are manual" section (the entrypoint auto-runs them now) and add a TLS section covering DATABASE_SSL/REDIS_TLS with the bundled plain Postgres/Redis.
- configuration reference: add DATABASE_SSL, REDIS_TLS, EMBEDDING_CACHE_DIR, the agent LLM provider, local embedding defaults, and clarify that GITHUB_PRIVATE_KEY is key contents (real newlines) belonging to the same App that is installed.
- llm-providers: add an Embeddings (PR memory) section documenting voyage vs local, dimensions, and the glibc-image requirement.
- faq + setup-walkthrough: troubleshooting for the crash-loop from production TLS defaults, the 404 installation-token error from mismatched App credentials, the empty-installation case, and "Indexed 0 comments" being expected on repos without review history.
@clearpr

clearpr Bot commented Jun 14, 2026

Copy link
Copy Markdown

ClearPR — Review failed: Unprocessable Entity: "Line could not be resolved" - https://docs.github.com/rest/pulls/reviews#create-a-review-for-a-pull-request.

1 similar comment
@clearpr

clearpr Bot commented Jun 14, 2026

Copy link
Copy Markdown

ClearPR — Review failed: Unprocessable Entity: "Line could not be resolved" - https://docs.github.com/rest/pulls/reviews#create-a-review-for-a-pull-request.

@clearpr

clearpr Bot commented Jun 14, 2026

Copy link
Copy Markdown

ClearPR — Review failed: Unprocessable Entity: "Line could not be resolved, Line could not be resolved, and Line could not be resolved" - https://docs.github.com/rest/pulls/reviews#create-a-review-for-a-pull-request.

@vineethkrishnan vineethkrishnan merged commit b53fa8d into main Jun 14, 2026
13 of 14 checks passed
@vineethkrishnan vineethkrishnan deleted the docs/deploy-learnings branch June 14, 2026 16:24
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