nfs icon indicating copy to clipboard operation
nfs copied to clipboard

Changing provisioned volume mode to 0777

Open hehesky opened this issue 3 years ago • 0 comments

NFS Provisioner: Changing provisioned volume mode to 0777

Description of your changes:

Changing provisioned volume mode to 0777 by adding a chmod call after the dir is created

This is to allow non-root consumer pod to gain access to the volume. It is quite common now that pods are running as non-root user (sometimes even with restricted SCC). The consumer pod should be responsible for preparing the volume (i.e. changing the permission to proper values)

I also added on replace clause in go.mod otherwise it won't build (see #39).

Which issue is resolved by this Pull Request: Resolves #

Resolves #22 and potentially a band-aid fix for #39

I've tested this change using a dev build running on Openshift 4.8 with portworx backing the server. With this change non-root pods are able to mount provisioned volume and gain access.

Checklist:

  • [ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • [ ] Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
  • [ ] Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
  • [ ] Reviewed the developer guide on Submitting a Pull Request
  • [ ] Documentation has been updated, if necessary.
  • [ ] Unit tests have been added, if necessary.
  • [ ] Integration tests have been added, if necessary.
  • [ ] Pending release notes updated with breaking and/or notable changes, if necessary.
  • [ ] Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
  • [ ] Code generation (make codegen) has been run to update object specifications, if necessary.

hehesky avatar Jan 27 '22 16:01 hehesky