docs(self-hosting): NodeLocal DNS and ClickHouse task events#3568
Conversation
|
WalkthroughThis PR updates self-hosting documentation to explain task event storage configuration. The changes introduce a new "Task events" section across Docker and Kubernetes documentation that describes default PostgreSQL storage and how to configure ClickHouse as an alternative via the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/self-hosting/kubernetes.mdx (1)
357-362: ⚡ Quick winAdd a concrete Helm snippet for
ndots: 1configuration.This section explains what to change, but not how to apply it in this chart. Adding a short
values.yamlexample forwebapp/supervisorpoddnsConfig.optionswould make the recommendation actionable.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/self-hosting/kubernetes.mdx` around lines 357 - 362, Add a concrete Helm values example showing how to set ndots to 1 for the webapp and supervisor pods: update the chart's values.yaml to include dnsConfig.options entries under the webapp and supervisor keys (referencing webapp and supervisor and dnsConfig.options) with an option name "ndots" and value "1", and mention that this applies to the pods' spec.dnsConfig to lower ndots from 5 to 1; keep the snippet minimal so readers can paste it into their values.yaml and deploy the chart.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@docs/self-hosting/kubernetes.mdx`:
- Around line 357-362: Add a concrete Helm values example showing how to set
ndots to 1 for the webapp and supervisor pods: update the chart's values.yaml to
include dnsConfig.options entries under the webapp and supervisor keys
(referencing webapp and supervisor and dnsConfig.options) with an option name
"ndots" and value "1", and mention that this applies to the pods' spec.dnsConfig
to lower ndots from 5 to 1; keep the snippet minimal so readers can paste it
into their values.yaml and deploy the chart.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 219fecd9-f520-4ae2-8c40-fbde86e90fd7
📒 Files selected for processing (3)
docs/self-hosting/docker.mdxdocs/self-hosting/env/webapp.mdxdocs/self-hosting/kubernetes.mdx
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: Mintlify Deployment
- GitHub Check: Analyze (actions)
- GitHub Check: Analyze (python)
- GitHub Check: Analyze (javascript-typescript)
🧰 Additional context used
📓 Path-based instructions (1)
docs/**/*.mdx
📄 CodeRabbit inference engine (docs/CLAUDE.md)
docs/**/*.mdx: MDX documentation pages must include frontmatter with title (required), description (required), and sidebarTitle (optional) in YAML format
Use Mintlify components for structured content: , , , , , , /, /
Always import from@trigger.dev/sdkin code examples (never from@trigger.dev/sdk/v3)
Code examples must be complete and runnable where possible
Use language tags in code fences:typescript,bash,json
Files:
docs/self-hosting/kubernetes.mdxdocs/self-hosting/env/webapp.mdxdocs/self-hosting/docker.mdx
🧠 Learnings (2)
📚 Learning: 2026-03-10T12:44:14.176Z
Learnt from: nicktrn
Repo: triggerdotdev/trigger.dev PR: 3200
File: docs/config/config-file.mdx:353-368
Timestamp: 2026-03-10T12:44:14.176Z
Learning: In the trigger.dev repo, docs PRs are often companions to implementation PRs. When reviewing docs PRs (MDX files under docs/), check the PR description for any companion/related PR references and verify that the documented features exist in those companion PRs before flagging missing implementations. This ensures docs stay in sync with code changes across related PRs.
Applied to files:
docs/self-hosting/kubernetes.mdxdocs/self-hosting/env/webapp.mdxdocs/self-hosting/docker.mdx
📚 Learning: 2026-04-30T20:30:29.458Z
Learnt from: ericallam
Repo: triggerdotdev/trigger.dev PR: 3226
File: docs/ai-chat/quick-start.mdx:13-13
Timestamp: 2026-04-30T20:30:29.458Z
Learning: In this repo’s documentation MDX files (`docs/**/*.mdx`), use `ts` and `tsx` (not `typescript`) as the code-fence language tags for TypeScript/TSX snippets. Do not flag `ts`/`tsx` code-fence language tags as incorrect in any docs MDX file, since this is the site-wide Mintlify-compatible convention.
Applied to files:
docs/self-hosting/kubernetes.mdxdocs/self-hosting/env/webapp.mdxdocs/self-hosting/docker.mdx
🪛 LanguageTool
docs/self-hosting/kubernetes.mdx
[grammar] ~361-~361: Ensure spelling is correct
Context: ... hosts). Lowering ndots to 1 on the webapp and supervisor pods avoids those extra ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[grammar] ~367-~367: Ensure spelling is correct
Context: ...EVENT_REPOSITORY_DEFAULT_STORE on the webapp via extraEnvVars: ```yaml webapp: ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
docs/self-hosting/docker.mdx
[grammar] ~349-~349: Ensure spelling is correct
Context: ...askEventtable. To enable, set on the webapp in your.env`: ```bash EVENT_REPOSITO...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
Summary
ndotsto1in the Kubernetes self-hosting guide.EVENT_REPOSITORY_DEFAULT_STORE=clickhouse_v2) in both the Docker and Kubernetes guides, plus a new row in the webapp env var reference.