container-storage-interface / spec

Container Storage Interface (CSI) Specification.
Apache License 2.0
1.34k stars 373 forks source link

Constraint of ephemeral volumes #557

Closed sasakiyori closed 10 months ago

sasakiyori commented 11 months ago

Persistent volumes can implement constraints by GetPluginCapabilities, CreateVolume and NodeGetInfo, which set the AccessibleTopology.

Can ephemeral volumes do this? It only calls NodePublishVolume but no CreateVolume. I don't know how to match accessible topologies with every CSI Node.

sasakiyori commented 11 months ago

It seems that topology for ephemeral volumes will not be supported: https://github.com/kubernetes/enhancements/tree/master/keps/sig-storage/596-csi-inline-volumes#non-goals

It there any alternative way to make constraints for non-persistent volumes?

jdef commented 10 months ago

you probably want to take this issue to k8s SIG storage, since they own the k8s implementation of CSI

On Wed, Nov 22, 2023 at 9:23 PM sasakiyori @.***> wrote:

It seems that topology for ephemeral volumes will not be supported: https://github.com/kubernetes/enhancements/tree/master/keps/sig-storage/596-csi-inline-volumes#non-goals

It there any alternative way to make constraints for non-persistent volumes?

— Reply to this email directly, view it on GitHub https://github.com/container-storage-interface/spec/issues/557#issuecomment-1823748952, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAR5KLHAOZMQVS4M3RAPZ53YF2XTDAVCNFSM6AAAAAA7XCDTM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRTG42DQOJVGI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- James DeFelice

sasakiyori commented 10 months ago

OK. Thanks