cloud-provider-openstack icon indicating copy to clipboard operation
cloud-provider-openstack copied to clipboard

cinder-CSI: support fsGroup in CSI

Open jichenjc opened this issue 2 years ago • 7 comments

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug /kind feature

What happened:

check https://kubernetes-csi.github.io/docs/support-fsgroup.html looks like CSI can support this fsGroup feature, not sure whether openstack user need this or not..

What you expected to happen:

How to reproduce it:

Anything else we need to know?:

Environment:

  • openstack-cloud-controller-manager(or other related binary) version:
  • OpenStack version:
  • Others:

jichenjc avatar Jan 12 '23 01:01 jichenjc

Hi,

just want to clarify, cinder currently does not support fsGroups, right? I've been trying to troubleshoot this all day

mogindi avatar Feb 16 '23 16:02 mogindi

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar May 17 '23 16:05 k8s-triage-robot

/remove-lifecycle stale

mdbooth avatar May 18 '23 09:05 mdbooth

I was having OpenStack provided volumes mounted as root:root and thinking it's me who missing something apart from using the fsGroup specified in a Pod's securityContext until someone pointed me at https://github.com/kubernetes/cloud-provider-openstack/blob/master/pkg/csi/cinder/nodeserver.go saying it's not implemented...

Ashame that this feature , that personally feels like really important one, is not implemented. I wish I would have any Golang or OpenStack experience to dive into implementation...

RoSk0 avatar Dec 08 '23 00:12 RoSk0

I've been also trying to make a volume to be mounted as other than 'root:root' and it didn't work so far. As @RoSk0 , I tried with fsGroup but that didn't make any difference. In my case, this problem happens when I create a PVC that consumes a storageclass designed for multiattach (ReadWriteMany PVCs). If the storageclass is the one that I use for ReadWriteOnce PVCs, the group that owns the volume can be different than root:root (apparently, it's set using the fsGroup, so if the fsGroup is 1001, the volume will be mounted as root:1001).

amgonzalezf avatar Jan 12 '24 16:01 amgonzalezf

I've been looking into this a bit and it looks like you can indeed use fsGroup successfully.

What caught us out, is having a ReadWriteMany accessModes set on the PVC, instead of obviously having to use ReadWriteOnce, i.e:

spec:
  storageClassName: b1.standard
  accessModes:
    - ReadWriteOnce

Hope this helps someone!

euven avatar Jan 31 '24 03:01 euven

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Apr 30 '24 08:04 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle rotten
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar May 30 '24 08:05 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue with /reopen
  • Mark this issue as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close not-planned

k8s-triage-robot avatar Jun 29 '24 08:06 k8s-triage-robot

@k8s-triage-robot: Closing this issue, marking it as "Not Planned".

In response to this:

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Reopen this issue with /reopen
  • Mark this issue as fresh with /remove-lifecycle rotten
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/close not-planned

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

k8s-ci-robot avatar Jun 29 '24 08:06 k8s-ci-robot