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

Cannot use labels in subDir #791

Open
mmelcot opened this issue Nov 13, 2024 · 2 comments
Open

Cannot use labels in subDir #791

mmelcot opened this issue Nov 13, 2024 · 2 comments

Comments

@mmelcot
Copy link

mmelcot commented Nov 13, 2024

What happened:
When creating a StorageClass, if one use labels of a PVC or PV in the subDir, it is not correctely interpreted.

What you expected to happen:
I expect the NFS CSI storage class to create a folder hierarchy in the NFS server, using the label foo from my pvc.
Instead, I got something like:

          my_namespace
              ${pvc.metadata.labels.foo}
                     my_pvc_name

i.e. that the labels are not correctly interpreted.

How to reproduce it:
Define a StorageClass in which you configured subDir with something like:
subDir: ${pvc.metadata.namespace}/${pvc.metadata.labels.foo}/${pvc.metadata.name}

And then create a PVC with:

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: my-pvc
  labels:
    foo: "bar"

Create a pod using that PVC, and check which folder hierarchy it creates in the NFS

Environment:

  • CSI Driver version: registry.k8s.io/sig-storage/nfsplugin:v4.9.0
  • Kubernetes version (use kubectl version):
Client Version: v1.31.2
Kustomize Version: v5.4.2
Server Version: v1.30.3+k3s1
  • OS (e.g. from /etc/os-release): Ubuntu 22.04.5 LTS
  • Kernel (e.g. uname -a): Linux 6.8.0-48-generic #48~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Mon Oct 7 11:24:13 UTC 2 x86_64 x86_64 x86_64 GNU/Linux
@andyzhangx
Copy link
Member

only following metadata is supported per the doc: https://github.com/kubernetes-csi/csi-driver-nfs/blob/master/docs/driver-parameters.md#subdir-parameter-supports-following-pvpvc-metadata-conversion

${pvc.metadata.name}
${pvc.metadata.namespace}
${pv.metadata.name}

@mmelcot
Copy link
Author

mmelcot commented Nov 25, 2024

ah ok, thanks @andyzhangx for the clarification.

Well, that would be great if that list could be extended in a future version of csi-driver-nfs...

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

No branches or pull requests

2 participants