Skip to content

Conversation

@cod3mas0n
Copy link
Contributor

Type of Change

  • Bug Fix
    • Add one missing . in volumes in docker-compose.yml in Define 4 storage volumes environment.
  • Refactor
    • Add a very simple docker-compose-simple.yml to deploy RustFS alongside Redis cache for objects without any observability and proxy services (profiles).
    • Also add RUSTFS_OBJECT_CACHE_TTL_SECS=300 and RUSTFS_OBJECT_CACHE_ENABLE=true cache envs for the new docker-compose.
    • Add volume permission fixer service (init container) named volume-permission-helper.

Summary of Changes

Add a simple Docker Compose file without any proxy and observability profiles.

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit

Additional Notes


Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md) and sign the CLA if this is your first contribution.

Signed-off-by: Ali Mehraji <a.mehraji75@gmail.com>
Signed-off-by: Ali Mehraji <a.mehraji75@gmail.com>
@CLAassistant
Copy link

CLAassistant commented Dec 7, 2025

CLA assistant check
All committers have signed the CLA.

@cod3mas0n cod3mas0n changed the title simple deployment docker-compose simple deployment via docker-compose Dec 7, 2025
@houseme houseme requested review from Copilot and majinghe December 8, 2025 04:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a simplified Docker Compose deployment configuration and fixes a critical volume syntax bug in the existing docker-compose.yml.

  • Corrects volume path expansion syntax from {0..3} to {0...3} in docker-compose.yml
  • Adds docker-compose-simple.yml for minimal RustFS deployment with object caching enabled
  • Includes a volume permission fixer service to handle ownership for multiple volumes

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docker-compose.yml Fixes volume syntax from {0..3} to {0...3} (two dots to three dots) for proper path expansion in RUSTFS_VOLUMES environment variable
docker-compose-simple.yml Adds new simplified Docker Compose configuration with RustFS, Redis (optional via profile), in-memory object cache enabled, and init container for volume permissions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  - object caching is in memory
  - remove redis service and volumes

Signed-off-by: Ali Mehraji <a.mehraji75@gmail.com>
@cod3mas0n
Copy link
Contributor Author

cod3mas0n commented Dec 8, 2025

@houseme Let me know if you need refactoring of the main docker-compose.yml; I can do another PR.

@majinghe
Copy link
Contributor

majinghe commented Dec 8, 2025

/LGTM

/cc @houseme

@houseme houseme merged commit 15c75b9 into rustfs:main Dec 8, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants