Migrate data to external Postgres and storage#297
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
Next review available in: 58 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR adds a migration guide for moving Plane from local PostgreSQL and MinIO containers to managed PostgreSQL and S3-compatible storage, plus a matching VitePress sidebar link. ChangesMigration Guide Documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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.
Actionable comments posted: 4
🤖 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.
Inline comments:
In `@docs/self-hosting/manage/migration/migrate-data-to-external-services.md`:
- Around line 73-79: The MinIO migration steps mix credential names by
referencing AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY in the prose but still
using MINIO_ROOT_USER and MINIO_ROOT_PASSWORD in the mc alias set example.
Update the migration guide so the command uses the same local MinIO credential
names described above, and make sure the placeholders in the shell example match
the variables readers should pull from plane.env.
- Around line 1-4: The page frontmatter is missing the required keywords field,
so update the markdown frontmatter to include it alongside title and
description. Use the existing frontmatter block at the top of
migrate-data-to-external-services.md and add a keywords entry that matches the
document topic so it satisfies the docs rule for docs/**/*.md pages.
- Around line 25-35: The migration steps currently contradict each other because
docker compose down already stops plane-db and plane-minio, so the note after
that command should be reworded for consistency. Update the wording in the
migration guide around the docker compose down and docker compose up -d plane-db
plane-minio sequence to clearly say whether the intent is to keep the
volumes/data intact or to bring the services back up specifically for export,
and make the instructions in that section align with the actual behavior of
docker compose down.
- Around line 110-116: The environment variable example uses invalid env-file
syntax for AWS_REGION, so update the migration docs to use standard key=value
formatting consistently. In the example block under the external services setup,
change the AWS_REGION entry to match the other variables and ensure the snippet
remains valid for .env and Compose env files.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 30802ed8-b54c-4c9d-8b72-29501bfd20a1
📒 Files selected for processing (2)
docs/.vitepress/config.mtsdocs/self-hosting/manage/migration/migrate-data-to-external-services.md
…ostgreSQL version compatibility, and update MinIO instructions
Description
Type of Change
Screenshots and Media (if applicable)
Test Scenarios
References
Summary by CodeRabbit