[4726] fix(docs): update sparse-checkout command to include DB-init SQL#4879
[4726] fix(docs): update sparse-checkout command to include DB-init SQL#4879Sanket2329 wants to merge 1 commit into
Conversation
|
@Sanket2329 is attempting to deploy a commit to the agenta projects Team on Vercel. A member of the Team first needs to authorize it. |
|
✅ Thanks @Sanket2329! This PR now meets the contribution requirements and has been reopened. A maintainer will review it soon. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughBoth self-host setup docs now expand the ChangesSelf-host sparse-checkout docs
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related issues
✨ 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 |
|
Hi @jp-agenta, @mmabrouk – this pull request is ready for review. It implements the agreed Option A (extending the sparse-checkout command to include the database initialization SQL paths) to resolve issue #4726, and has been successfully verified locally. Could you please take a look when you have a moment? Thank you! |
Summary
Resolves #4726. When following the self-hosting Quick Start or Remote Deployment guides verbatim, a brand-new OSS deploy comes up with no application databases because the
docker-compose.gh.ymlfile bind-mountsapi/oss/databases/postgres/init-db-oss.sqlwhich lies outside the checked outhosting/docker-composesparse cone.This PR extends the
git sparse-checkoutcommands in the quick-start and remote deployment documentation to also fetch theapi/oss/databases/postgresandapi/ee/databases/postgresdirectories, ensuring that database initialization SQL scripts are present on disk before running the containers.Testing
Verified locally