-
-
Notifications
You must be signed in to change notification settings - Fork 998
docs: add ClickHouse migration troubleshooting tip #3011
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
WalkthroughThis change updates the self-hosting Docker troubleshooting docs: it replaces the previous guidance about "ERROR: schema "graphile_worker" does not exist" with instructions for a ClickHouse/goose migration mismatch where migrations report "no migrations to run" but the schema is missing. It instructs to exec into the webapp container, set the GOOSE environment variables, and run Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
🧹 Recent nitpick comments
📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📓 Common learnings📚 Learning: 2026-01-15T11:50:06.067ZApplied to files:
⏰ 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). (1)
✏️ Tip: You can disable this entire section by setting 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/self-hosting/docker.mdx`:
- Line 359: Update the ClickHouse migration troubleshooting note that suggests
running "goose reset && goose up" to include a clear, prominent data-loss
warning: explain that "goose reset" is destructive and drops schemas/tables,
instruct users to back up their ClickHouse data or confirm they are on a
non-production environment before running it, and recommend alternative steps
(inspect migration status, export schema/data, or test on a staging instance)
alongside the existing instruction to exec into the webapp container, set the
GOOSE environment variables from the webapp startup logs, and then run the goose
commands.
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/self-hosting/docker.mdx
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-01-15T11:50:06.067Z
Learning: Applies to internal-packages/clickhouse/schema/**/*.sql : ClickHouse migrations must use Goose format with `-- +goose Up` and `-- +goose Down` markers
📚 Learning: 2026-01-15T11:50:06.067Z
Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-01-15T11:50:06.067Z
Learning: Applies to internal-packages/clickhouse/schema/**/*.sql : ClickHouse migrations must use Goose format with `-- +goose Up` and `-- +goose Down` markers
Applied to files:
docs/self-hosting/docker.mdx
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
Adds troubleshooting tip for when ClickHouse migrations report "no migrations to run" but the schema is missing. This happens when the goose migration tracker is out of sync with the actual schema state.