feat: update OceanBase docker configuration #4
Conversation
- Switch from oceanbase-ce to seekdb image - Update OCEANBASE_USER to use root instead of root@tenant - Add healthcheck for powerrag service - Update OceanBase healthcheck configuration - Add .gitignore entries for oceanbase data and conf directories
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the OceanBase configuration from the oceanbase-ce image to the seekdb image, simplifying the authentication model and updating various container configurations.
Key Changes:
- Switched from
oceanbase/oceanbase-ce:4.4.1.0-100000032025101610tooceanbase/seekdb:latest - Updated authentication from tenant-based (
root@tenant) to simple root user - Added healthcheck monitoring for the powerrag service with dual endpoint validation
Reviewed Changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docker/docker-compose.yml | Migrated oceanbase service to seekdb image, updated volume paths, modified healthcheck configuration, and added powerrag service healthcheck |
| docker/.env.example | Changed OCEANBASE_USER from tenant-based to simple root authentication |
| .gitignore | Added exclusions for new oceanbase data and conf directories matching updated volume mount paths |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
whhe
left a comment
whhe
left a comment
There was a problem hiding this comment.
We need also to update the readme file in docker directory for this change, and I'm not sure if the seekdb container needs to set '127.0.0.1' as IP like langgenius/dify#20103
- Simplify ROOT_PASSWORD environment variable configuration - Add OB_SERVER_IP environment variable for explicit server binding - Increase healthcheck intervals and retries for more reliable startup - interval: 5s -> 10s - retries: 5 -> 30 - start_period: 10s -> 30s - Improve OCEANBASE_USER configuration documentation in .env.example
2 participants
Summary
Solution Description