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.

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

Fix the problem in "CSI volume cloning" document #16321

Closed
draghuram opened this issue Sep 11, 2019 · 2 comments · Fixed by #16521
Closed

Fix the problem in "CSI volume cloning" document #16321

draghuram opened this issue Sep 11, 2019 · 2 comments · Fixed by #16521
Assignees
@draghuram
Labels
language/en Issues or PRs related to English language priority/backlog Higher priority than priority/awaiting-more-evidence.

Comments

@draghuram
Copy link
Contributor

This is a Bug Report

Problem:
The page at:
https://kubernetes.io/docs/concepts/storage/volume-pvc-datasource/
states:
"The implementation of cloning, from the perspective of the Kubernetes API simply adds the ability to specify an existing unbound PVC as a dataSource during new pvc creation."

I think "unbound PVC" is incorrect as without binding, there is no volume that can be cloned.

Proposed Solution:

It seems to me that changing "unbound" to "bound" should fix the issue.

Page to Update:
https://kubernetes.io/docs/concepts/storage/volume-pvc-datasource/
draghuram pushed a commit to draghuram/website that referenced this issue Sep 16, 2019
Fixed few issues with Storage concepts section.
1bc263d 
- Removed plural api object names such as `PersistentVolumes`.

- Explicitly mentioned alpha status of snapshot APIs.

- Fixed kubernetes#16321.

- Other minor changes.
@draghuram draghuram mentioned this issue Sep 16, 2019
Closed
@sftim
Copy link
Contributor

sftim commented Sep 17, 2019

/language en
/priority backlog

@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language priority/backlog Higher priority than priority/awaiting-more-evidence. labels Sep 17, 2019
@draghuram
Copy link
Contributor Author

/assign draghuram

@draghuram draghuram mentioned this issue Sep 23, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@draghuram draghuram

Labels
language/en Issues or PRs related to English language priority/backlog Higher priority than priority/awaiting-more-evidence.
Projects
None yet
Milestone
No milestone
3 participants
@draghuram @k8s-ci-robot @sftim