Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce file permissions #3865

Merged
merged 1 commit into from
Feb 7, 2025
Merged

Conversation

fengwei0328
Copy link
Contributor

@fengwei0328 fengwei0328 commented Feb 6, 2025

nerdctl                                                                                    drwx------.
└── 1935db59                                                                               drwx------.
    ├── containers                                                                         drwx------.
    │   └── k8s.io                                                                         drwx------.
    │       ├── 1b0eced10dea9fddab7c66631f0920170121bd996d2845b48468ff119fe40826           drwx------.
    │       │   ├── 1b0eced10dea9fddab7c66631f0920170121bd996d2845b48468ff119fe40826-json.log    -rw-------.
    │       │   ├── hostname                                                               -rw-r--r--.    //same as the host path
    │       │   ├── log-config.json                                                        -rw-------.
    │       │   ├── oci-hook.createRuntime.log                                             -rw-r--r--.    
    │       │   └── resolv.conf                                                            -rw-r--r--.    //same as the host path
    │       └── 398edb6ec491ebbbe5175e9ce7acfadaabac72c4f6de07995d685f452cddae4d           drwx------.
    │           ├── 398edb6ec491ebbbe5175e9ce7acfadaabac72c4f6de07995d685f452cddae4d-json.log    -rw-------.
    │           └── log-config.json                                                        -rw-------.
    ├── etchosts                                                                           drwx------.
    │   └── k8s.io                                                                         drwx------.
    │       └── 1b0eced10dea9fddab7c66631f0920170121bd996d2845b48468ff119fe40826           drwx------.
    │           ├── hosts                                                                  -rw-r--r--.    //same as the host path
    │           └── meta.json                                                              -rw-r--r--.    //644 -> 640
    ├── names                                                                              drwx------.
    │   └── k8s.io                                                                         drwx------.
    │       └── busybox-1b0ec                                                              -rw-------.
    └── volumes                                                                            drwx------.
        └── k8s.io                                                                         drwx------.                                                   
            ├── 48c16a5d59bdaa86a12e39f9f926a2b8d1a79c88545126b43df07a298aa45bc3           drwx------.
            │   ├── _data                                                                  drwxr-xr-x.    //same as the volume-mount path
            │   └── volume.json                                                            -rw-r--r--.    //644 -> 640
            └── 52caf01568a036a1cae077fbd5a8fd5d1c50bc45f1df022bf8e89c1c400ee465           drwx------.
                ├── _data
                └── volume.json

@fengwei0328 fengwei0328 force-pushed the permission branch 10 times, most recently from 9af9c13 to 2503b1a Compare February 7, 2025 02:57
Signed-off-by: fengwei0328 <feng.wei8@zte.com.cn>
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda added this to the v2.x.x (tentative) milestone Feb 7, 2025
@AkihiroSuda AkihiroSuda merged commit 73a4b01 into containerd:main Feb 7, 2025
30 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