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

Backend probelm after upgrade to v4.0.0 #125

Open
pbabilas opened this issue May 2, 2023 · 2 comments
Open

Backend probelm after upgrade to v4.0.0 #125

pbabilas opened this issue May 2, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@pbabilas
Copy link

pbabilas commented May 2, 2023

As in topic, when I updated my Hua driver at cluster, there is a problem with backend names. In V3 backend name is set like: backend: dorado-3000-V6 - during this version there was only string type restriction in this field. Now at v4 I cannot use this name because crd meta.name which is created from backend needs to by lowercased :(

Invalid value: "test-Name": a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

My cluster has more than 6 hundreds lun created, all with backend name writed before. Background name can be changed in storageclass, in configmap and secret but pv is immutable and to do so I would need to scale all services, delete pvc and delete pv, next again add those with backend name changed.
Is there any way that driver could convert name to lowercase or work with backward compatibility?

@ws410
Copy link
Collaborator

ws410 commented May 8, 2023

We have noticed this issue. In version 4.0.0, we require the backend name to follow the DNS label standard defined in RFC 1123,before version 4.0.0, we did not have this requirement, and there are indeed differences. Please let me know which version of CSI you are currently using, so that we can better solve the issue.

@pbabilas
Copy link
Author

pbabilas commented May 8, 2023

Thanks for answer, my current version os 3.1.

@YeYoot YeYoot added the bug Something isn't working label May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants