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

[CSI] Profile name should be supported in place of profile id in storage class #334

Open
sushanthakumar opened this issue Jan 19, 2020 · 2 comments
Labels

Comments

@sushanthakumar
Copy link
Collaborator

Is this a BUG REPORT or FEATURE REQUEST?:

/kind feature

What happened:
At present, CSI takes profile info through profile id. Profile id can change and also not convenient for user to use. So profile name is more useful and convenient to use while creating K8S storage class

image

What you expected to happen:
CSI plugin should support accepting profile name instead of profile id

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • NBP version:
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:
@sushanthakumar
Copy link
Collaborator Author

Not addressing for Faroe release

@asifdxtreme
Copy link
Member

@sushanthakumar @kumarashit As per Isabela relase, the Dashboard allows changes to the name of soda-profile, so in this case it will be contradictory.
So I guess the ID is constant and the name can change, so IMO let's better stick to profile ID OR else we disable the edit option for soda-profile name and then make the name as the parameter in storage class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants