-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Adds documentation for Dell EMC ScaleIO volume plugin #2741
Adds documentation for Dell EMC ScaleIO volume plugin #2741
Conversation
* `protectionDomain`: the name of the ScaleIO protection domain | ||
* `storagePool`: the name of the volume storage pool | ||
* `storageMode`: the way storage is provisioned (ThinProvisionned or ThickProvisionned) | ||
* `secretRef`: name reference to a configuered Secret object |
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.
- In what namespace is the secret expected?
- What are expected keys and values of the secret? Please follow example e.g. from quobyte above.
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.
@jsafrane Hi, I updated the doc to address your comments about namespace values.
#### ScaleIO | ||
```yaml | ||
kind: StorageClass | ||
apiVersion: storage.k8s.io/v1beta1 |
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.
Please use storage.k8s.io/v1
, I'm waiting for #2686 to get merged.
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.
Done!
This commit updates the volumes documentation to include ScaleIO volume plugin information.
Thanks for the PR. |
@jsafrane thank for the review. |
@jsafrane is there anything else I need to do for this to get merged? Thanks. |
Someone from docs team should merge it, I do just technical review. To be honest, I don't understand why it's assigned to me. @chenopis, PTAL. |
This commit updates the volumes documentation to include ScaleIO volume plugin information.
This change is