wunderio / csi-rclone

CSI driver for rclone
Apache License 2.0
214 stars 62 forks source link

added support for secretName in PV volumeAttributes to override the default #7

Open cameronbraid opened 4 years ago

cameronbraid commented 4 years ago

I'd also like it to look for the secret in the same namespace as the PV/PVC

Is there any way to get this info ?

Jancis commented 4 years ago

Can't read secret from another namespace, unfortunately

Secret resources reside in a namespace . Secrets can only be referenced by Pods in that same namespace. https://kubernetes.io/docs/concepts/configuration/secret/#details

dvcrn commented 2 years ago

This PR is 2 years old at this point, but I went ahead and merged it into my fork https://github.com/dvcrn/csi-rclone-reloaded

Hope upstream will reconsider it

Jancis commented 2 years ago

I was under impression this does not work due to technical limitation. Should probably test it. I love the fork and examples in readme :+1: There is not much work done on this for a while as we've been using filestore and trying out nfs provisioner with hosted nfs server. It handles plugin container restarts way better. Still, we have few ideas to change about this module in the backlog and since CSI spec seems to have changed a bit it will require at least partial rewrite anyway.

dvcrn commented 2 years ago

I was under impression this does not work due to technical limitation. Should probably test it.

You just made me retry this to make sure 😄 The secret needs in the same namespace as the controller, then everything is fine. So in my fork that would be kube-system

Let me add it to the README

danielloader commented 1 year ago

This PR is 2 years old at this point, but I went ahead and merged it into my fork https://github.com/dvcrn/csi-rclone-reloaded

Hope upstream will reconsider it

@dvcrn Just found this but your fork doesn’t include issues. Fancy merging https://github.com/wunderio/csi-rclone/pull/46 too?

dvcrn commented 1 year ago

Oh, Issues were not supposed to be off, I've enabled them.

I'll put it on my todo list but may take a bit until I get to it!