-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add some parameters to change volumeNamePrefix
with csi-external-provisioner
#87
Comments
The volumeNamePrefix argument is not that useful. If you change the prefix from "pvc-" to let's say "abc-", then the name of all PVs will have "abc-" as prefix. -Subodh |
Yes, that would be the function I want to do. If I have 2 Nutanix Volume, then it would have different prefix as sparate, like On the other hand, If I have multiple k8s cluster to share with same Nutanix Volume, it would be named as PV Name, like Like above, |
Okay, this is a valid use case. We can expose volumeNamePrefix customization via helm chart. @tuxtof |
Hi @rockwyc992 we have the field Description who allow to add some info in the VG to identify your cluster for exemple |
Yes, I use that field to identify which volume is belong to corresponding kubernetes cluster. But this is not easy to search and maintain. With this situation, I must to click and open every volume in the Prism console to identify. |
i know and i agree |
Are there any method to change
volumeNamePrefix
for the new PV Name ?If there would be multiple Nutanix Volume to use, every PV Name are same with the prefix
pvc-
by default.Maybe adding some
args
forcsi-external-provisioner
is a good idea. like below:https://github.com/nutanix/helm/blob/master/charts/nutanix-csi-storage/templates/ntnx-csi-controller-deployment.yaml#L44
And the default parameter to use for
volumeNamePrefix
like below:https://github.com/nutanix/csi-external-provisioner/blob/master/cmd/csi-provisioner/csi-provisioner.go#L62
https://github.com/nutanix/csi-external-provisioner/blob/master/pkg/controller/controller.go#L558
The text was updated successfully, but these errors were encountered: