search

kubernetes / cloud-provider-openstack

Apache License 2.0
619 stars 610 forks source link

doesn't support volume2 any more? #2676

Open PilotPaul opened 1 month ago

PilotPaul commented 1 month ago

https://github.com/kubernetes/cloud-provider-openstack/blob/e951ec3824631820d2a6527374e036fe33d58d73/pkg/csi/cinder/openstack/openstack_volumes.go#L116

PilotPaul commented 1 month ago

The following code changes look more reasonable and support v2 and v3 versions:

blockstorageClient := *os.blockstorage if !os.bsOpts.IgnoreVolumeMicroversion { blockstorageClient.Microversion = "3.34" }

opts := volumes.ListOpts{Name: n} pages, err := volumes.List(&blockstorageClient, opts).AllPages()

kayrus commented 1 month ago

@PilotPaul there was never support for v2/volumes in the repo.

PilotPaul commented 1 month ago

@PilotPaul there was never support for v2/volumes in the repo.

got it, thx a lot