Skip to content

Conversation

@majinghe
Copy link
Contributor

@majinghe majinghe commented Nov 7, 2025

Type of Change

  • New Feature
  • Bug Fix
  • Documentation
  • Performance Improvement
  • Test/CI
  • Refactor
  • Other:

Related Issues

#804

Summary of Changes

Modify Dockerfile to add non root user part. Testing result is as below,

for docker container deployment

docker exec -it 27cd798ca835 /bin/sh
/ $ id
uid=1000(rustfs) gid=1000(rustfs) groups=1000(rustfs)
/ $ whoami
rustfs
/ $ ls -ld /data/
drwxr-x--- 3 rustfs rustfs 4096 Nov  7 12:48 /data/
/ $ ls -ld /logs/
drwxr-xr-x 2 rustfs rustfs 4096 Nov  7 13:06 /logs/
/ $ ps -ef
PID   USER     TIME  COMMAND
    1 rustfs    0:00 /usr/bin/rustfs /data
   52 rustfs    0:00 /bin/sh
   69 rustfs    0:00 ps -ef
/ $ 

for k8s deployment

 $ id
uid=1000(rustfs) gid=1000(rustfs) groups=1000(rustfs)
/ $ whoami
rustfs
/ $ ls -ld /data/
drwxr-x--- 6 rustfs rustfs 4096 Nov  7 13:10 /data/
/ $ ls -d /logs/
/logs/
/ $ ls -ld /logs/
drwxr-xr-x 2 rustfs rustfs 4096 Nov  7 13:10 /logs/

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit
  • Added/updated necessary tests
  • Documentation updated (if needed)
  • CI/CD passed (if applicable)

Impact

  • Breaking change (compatibility)
  • Requires doc/config/deployment update
  • Other impact:

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.

@loverustfs loverustfs merged commit 4716454 into rustfs:main Nov 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.

2 participants