Suggestion: Unify directory paths in Docker image for better non-root user support
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 8.5k
- Forks
- 1.3k
- Avg merge
- 5h 36m
- Merged PRs (30d)
- 22
Description
I would like to suggest an improvement to the official Docker image's directory structure to better support non-root users.
Current Situation
The Docker image appears to use /root/Megatron-LM and /sgl-workspace directories, which can cause permission issues for non-root users trying to access these paths.
Problem
/root/directory is only accessible by the root user- Non-root users cannot easily access files in these locations
- This creates friction when users want to run containers as non-root for security reasons
Suggested Solution
Relocate the directories to a standard location like:
/opt/Megatron-LMand/opt/sgl-workspace, or/workspace/Megatron-LMand/workspace/sgl-workspace
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the Docker image definitions and every reference to /root/Megatron-LM and /sgl-workspace. Check how those paths are created and used, then verify that the selected standard location is accessible to a non-root container user without breaking existing workflows.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker
- Domain
- devops
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100