Skip to content

RFC: refactor persistent storage example and add Cinder documentation#4403

Merged
openshift-bot merged 3 commits intoopenshift:masterfrom
jsafrane:devel/cinder-doc
Oct 16, 2015
Merged

RFC: refactor persistent storage example and add Cinder documentation#4403
openshift-bot merged 3 commits intoopenshift:masterfrom
jsafrane:devel/cinder-doc

Conversation

@jsafrane
Copy link
Contributor

These two patches modify the Wordpress example to include documentation of Cinder persistent volumes. Additional storage plugins can be added easily later. Is this something that you would be interested in? Is the structure fine?

Cinder support is heading to upstream Kubernetes in [PR #6689] kubernetes/kubernetes#6689). Kubernetes changes have been merged in openshift/origin commit 2009660

@markturansky
Copy link
Member

At first glance, this looks like a great addition to the wordpress example that shows how to change out the PV part in the backend. I'll take a more careful/thorough look later.

@jsafrane
Copy link
Contributor Author

Note that I had to disable SELinux to make it working - newly created ext3 filesystems are not labelled (system_u:object_r:unlabeled_t) and SELinux denies containers (svirt_lxc_net_t) to write to them. This is not Cinder related, it provides block device, not the filesystem. All other block-device-like volumes (iSCSI, RBD, ...) will have the same problem. Is the user supposed to label the filesystems inside the volumes?

@danmcp
Copy link

danmcp commented Sep 15, 2015

@markturansky @childsb Review please.

@childsb
Copy link
Contributor

childsb commented Sep 23, 2015

LGTM

@jsafrane
Copy link
Contributor Author

Rebased. Also, Kubernetes PR for OpenStack/Cinder was merged and it will land in OpenShift with next Kubernetes rebase.

Move the NFS docs to standalone document and leave the common parts to
be reused by documentation of other volume plugins.
Signed-off-by: Huamin Chen <hchen@redhat.com>
@danmcp
Copy link

danmcp commented Oct 16, 2015

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/5856/) (Image: devenv-fedora_2480)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to a81be3f

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/5856/)

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to a81be3f

openshift-bot pushed a commit that referenced this pull request Oct 16, 2015
@openshift-bot openshift-bot merged commit 5c9ecf6 into openshift:master Oct 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants