Unverified Commit 3af06ccf authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #54581 from bsnchan/conformance_host_path_permissions

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Removes 'rwx' permissions for global users - the tests make an assumption that the permissions on the /tmp dir have not been altered Signed-off-by: 's avatarBrenda Chan <brchan@pivotal.io> **What this PR does / why we need it**: This PR modifies a conformance test that checks the file permissions when the`/tmp` dir is mounted. The current tests make an assumption that the permissions on the `/tmp` dir on the host system has not been altered. We removed the check that global users need `rwx`, so the tests now only check for `dtrwxrwx` **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: N/A **Special notes for your reviewer**: N/A **Release note**: ```release-note NONE ```
parents 18402f6c d7768e62
...@@ -51,7 +51,7 @@ var _ = Describe("[sig-storage] HostPath", func() { ...@@ -51,7 +51,7 @@ var _ = Describe("[sig-storage] HostPath", func() {
fmt.Sprintf("--file_mode=%v", volumePath), fmt.Sprintf("--file_mode=%v", volumePath),
} }
f.TestContainerOutput("hostPath mode", pod, 0, []string{ f.TestContainerOutput("hostPath mode", pod, 0, []string{
"mode of file \"/test-volume\": dtrwxrwxrwx", // we expect the sticky bit (mode flag t) to be set for the dir "mode of file \"/test-volume\": dtrwxrwx", // we expect the sticky bit (mode flag t) to be set for the dir
}) })
}) })
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment