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.

Sign up for GitHub

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

Jump to bottom

deviceClass to storage class mapping #40

Closed

leelavg opened this issue Dec 22, 2021 · 0 comments · Fixed by #47

Contributor

leelavg commented Dec 22, 2021 •

edited

Loading

there's a possibility that deviceClass name may not conform to dns naming convention
in that scenario directly mapping it to the storage class name may fail, need to handle that scenario
ref: feat: topolvm storage class resource unit implementation #38 (comment)

leelavg mentioned this issue

fix: validate deviceClass name to be usable in storageClass #47

Merged

nbalacha closed this as completed in #47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment