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

PR for adding ScaleIO PV documentation in volumes.md #2845

Merged
merged 1 commit into from
Mar 27, 2017
Merged

PR for adding ScaleIO PV documentation in volumes.md #2845

merged 1 commit into from
Mar 27, 2017

Conversation

vladimirvivien
Copy link
Member

@vladimirvivien vladimirvivien commented Mar 15, 2017

This PR adds PV documentation to volumes.md for ScaleIO volumes. Documentation for PVCs were done in #2741.


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 15, 2017
@vladimirvivien
Copy link
Member Author

@mikedanese Hi, any chance someone can take a look at this PR. Thanks.

@mikedanese mikedanese assigned saad-ali and unassigned mikedanese Mar 20, 2017
@chenopis
Copy link
Contributor

@vladimirvivien FYI we moved Volumes to /docs/concepts/storage/volumes.md, which is why there is a merge conflict. If you can apply your changes to the document in the new location, I can merge it after a tech review lgtm.

@chenopis chenopis added this to the 1.6 milestone Mar 22, 2017
ScaleIO is a software-based storage platform that can use existing hardware to create clusters of scalable
shared block networked storage. The ScaleIO volume plugin allows deployed pods to access existing ScaleIO
volumes (or it can dynamically provision new volumes for persistent volume claims, see
[ScaleIO Persistent Volumes](./persistent-volumes/index.md#scaleio)).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this document has moved, you'll probably want to use a less relative URL for the link:

/docs/user-guide/persistent-volumes/#scaleio

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chenopis the doc has been moved to concepts/docs/storage and the relative links fixed. Looking for LGTM. Thanks.

@vladimirvivien
Copy link
Member Author

@chenopis @mikedanese this PR is ready for review, looking for LGTM for merge. Thanks.

@mikedanese mikedanese removed their request for review March 25, 2017 17:10
@mikedanese
Copy link
Member

Tech LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants