-
Notifications
You must be signed in to change notification settings - Fork 388
Automatically create a dynamically provisioning StorageClass #484
base: master
Are you sure you want to change the base?
Automatically create a dynamically provisioning StorageClass #484
Conversation
|
Can one of the admins verify this patch? |
jarrpa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! I don't think we need to bother having two flags. Just use one --storage-class flag, where it can be in one of three states:
- Default value of
"", no SC - If flag is specified but with no value, default value of
glusterfs - If flag is specified with a value, use that as the SC name.
|
OK, sure. I can make those changes. |
jarrpa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Close, but not quite. I want the default value at the top of the file to be "". If the option is specified, that value changes to "glusterfs", and if the option is specifies a value then that value is used as the StorageClass name.
|
OK, I messed up on the last one, I get what you mean now. I'll fix it in a day or two, I need to make another test cluster to validate it on first... Also I was thinking I could add an option to set the StorageClass as default, your thoughts? |
|
Couldn't hurt to add it. |
|
How about a --storage-class-volumetype option as well? For minikube I would need volumetype: "none", and it would be nice to use this new feature. |
Add option to automatically create a dynamically provisioning StorageClass after the GlusterFS daemon-set deployment has completed.
To automatically create a StorageClass named:
glusterfs-storageTo automatically create a StorageClass named:
glusterfs-ssd-storageI don't have access to an OpenShift cluster to test this against so I didn't include the template for OpenShift.
This change is