fix(podman): relax image tar directory permissions for WSL#52215
fix(podman): relax image tar directory permissions for WSL#52215chienchandler wants to merge 1 commit intoopenclaw:mainfrom
Conversation
The setup script creates a temp directory for the Docker image tar with chmod 700, which prevents the openclaw user from reading the file during podman image load on Windows WSL. Change to 755 so the directory is world-readable while the owner retains full control. Fixes openclaw#52180
Greptile SummaryThis PR fixes a WSL-specific permission issue in
Confidence Score: 5/5
Reviews (1): Last reviewed commit: "fix(podman): relax image tar directory p..." | Re-trigger Greptile |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a91c223d1a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
CI note: Both failing checks fail on |
|
Closing because this targets the old tar-based Podman setup path, which we no longer use on current |
What
scripts/podman/setup.shcreates the image tar temp directory withchmod 700, which blocks the openclaw user from reading the tar file duringpodman image loadon Windows WSL.Fixes #52180
How
chmod 700→chmod 755. The directory is temporary (cleaned up on EXIT trap) and only holds a build artifact, so world-readable is fine here.Testing
Verified the permission change is consistent with how other temp directories are handled in the script. The directory is short-lived and removed by the cleanup trap at line 256.
AI-assisted (Claude Code), reviewed by author.