OSDOCS-9364 updated ROSA PV Storage options #70909
Conversation
|
/retest |
|
🤖 Mon Feb 05 22:12:16 - Prow CI generated the docs preview: https://70909--ocpdocs-pr.netlify.app |
openshift-ci
bot
added
the
size/S
xJustin
force-pushed
the
OSDOCS-9364-ROSA-storage-options
branch
4 times, most recently
from
87908f8 to
3324958
Compare
|
|
||
| ifdef::openshift-rosa[] | ||
| [id="types-of-persistent-volumes_{context}"] | ||
| == Types of PVs |
There was a problem hiding this comment.
For the Types of PVs we generally consider it as the volume type is Filesystem or Block . Here it seems means the csi drivers(different csi provisioners) using the title maybe a bit mislead since you also mentioned that ROSA also functions with Kubernetes Container Storage Interface (CSI) compatible volume provisioners from third-party storage vendors. We could change to e.g. Supported CSI drivers maybe better.
Ref: https://docs.openshift.com/container-platform/4.14/storage/container_storage_interface/persistent-storage-csi.html#csi-drivers-supported_persistent-storage-csi
cc to @xueli181114 @duanwei33
There was a problem hiding this comment.
@Phaow I have no problem changing the title. I believe it may be more accurate to mention the CSI drivers. what about something like:
== Supported CSI drivers for PVs or something to that effect >?
There was a problem hiding this comment.
Hello @xJustin , seems the enterprise still use the Types of PVs, I think we could keep it firstly the enterprise doc also will do some more updates in next releases, in origin upstream docs Types of volumes maybe more clearly. Anyway we could keep it for now and update with the enterprise docs together later.
https://kubernetes.io/docs/concepts/storage/volumes/#awselasticblockstore
| - AWS Elastic Block Store (EBS) | ||
| - AWS Elastic File Store (EFS) | ||
|
|
||
| ROSA also functions with Kubernetes Container Storage Interface (CSI) compatible volume provisioners from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for Red Hat OpenShift Service on AWS] matrix for more information. |
There was a problem hiding this comment.
From my understanding, we'd better move this description to https://70909--ocpdocs-pr.netlify.app/openshift-rosa/latest/storage/container_storage_interface/persistent-storage-csi#csi-drivers-supported_persistent-storage-csi if we want to keep consistent with the enterprise docs(keep using Types of PVs). The Types of PVs in enterprise docs specified the intree manual PVs not the CSI.
There was a problem hiding this comment.
@arendej what do you think about moving the description to the CSI page and linking over to it?
xJustin
force-pushed
the
OSDOCS-9364-ROSA-storage-options
branch
2 times, most recently
from
4935af2 to
eb040ec
Compare
|
LGTM |
xJustin
force-pushed
the
OSDOCS-9364-ROSA-storage-options
branch
from
eb040ec to
ae0be86
Compare
xJustin
force-pushed
the
OSDOCS-9364-ROSA-storage-options
branch
from
ae0be86 to
a16f3e0
Compare
|
/label peer-review-needed |
openshift-ci
bot
added
the
peer-review-needed
|
/label peer-review-in-progress |
openshift-ci
bot
added
the
peer-review-in-progress
| [IMPORTANT] | ||
| ==== | ||
| If your CSI driver is not listed in the preceding table, you must follow the installation instructions provided by your CSI storage vendor to use their supported CSI features. | ||
| ==== | ||
| endif::openshift-rosa[] | ||
| ifdef::openshift-rosa[] | ||
| In addition to the drivers listed in the preceding table, ROSA functions with CSI drivers from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for Red Hat OpenShift Service on AWS] matrix for more information. |
There was a problem hiding this comment.
Tiny nit - use {nbsp} in between Red and Hat so the company name is never broken up by a new line:
| In addition to the drivers listed in the preceding table, ROSA functions with CSI drivers from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for Red Hat OpenShift Service on AWS] matrix for more information. | |
| In addition to the drivers listed in the preceding table, ROSA functions with CSI drivers from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for Red{nbsp}Hat OpenShift Service on AWS] matrix for more information. |
|
/remove-label peer-review-needed /label peer-review-done |
openshift-ci
bot
added
peer-review-done
xJustin
force-pushed
the
OSDOCS-9364-ROSA-storage-options
branch
from
a16f3e0 to
5b5aacd
Compare
|
/label merge-review-needed |
openshift-ci
bot
added
the
merge-review-needed
jeana-redhat
added
the
merge-review-in-progress
| [IMPORTANT] | ||
| ==== | ||
| If your CSI driver is not listed in the preceding table, you must follow the installation instructions provided by your CSI storage vendor to use their supported CSI features. | ||
| ==== | ||
| endif::openshift-rosa[] | ||
| ifdef::openshift-rosa[] | ||
| In addition to the drivers listed in the preceding table, ROSA functions with CSI drivers from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for Red{nbsp}Hat OpenShift Service on AWS] matrix for more information. |
There was a problem hiding this comment.
| In addition to the drivers listed in the preceding table, ROSA functions with CSI drivers from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for Red{nbsp}Hat OpenShift Service on AWS] matrix for more information. | |
| In addition to the drivers listed in the preceding table, ROSA functions with CSI drivers from third-party storage vendors such as AWS FSX or Pure Storage Portworx. Red Hat does not oversee third-party provisioners or the connected CSI drivers and the vendors fully control source code, deployment, operation, and Kubernetes compatibility. These volume provisioners are considered customer-managed and the respective vendors are responsible for providing support. See the link:https://docs.openshift.com/rosa/rosa_architecture/rosa_policy_service_definition/rosa-policy-responsibility-matrix.html#rosa-policy-responsibilities_rosa-policy-responsibility-matrix[Shared responsibilities for {product-title}] matrix for more information. |
There was a problem hiding this comment.
Thanks @jeana-redhat! Updated! this one is good to merge whenever you get the chance
openshift-ci
bot
removed
merge-review-in-progress
xJustin
force-pushed
the
OSDOCS-9364-ROSA-storage-options
branch
from
5b5aacd to
81a8489
Compare
|
@xJustin: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
/label merge-review-needed |
|
/cherrypick enterprise-4.15 |
|
/cherrypick enterprise-4.14 |
|
/cherrypick enterprise-4.13 |
openshift-ci
bot
added
the
merge-review-needed
jeana-redhat
removed
the
merge-review-needed
|
@jeana-redhat: new pull request created: #71243 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@jeana-redhat: new pull request created: #71244 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@jeana-redhat: new pull request created: #71245 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Updated the ROSA PV storage options to more accurately reflect other providers.
Version(s):
4.13+
Issue:
https://issues.redhat.com/browse/OSDOCS-9364
Link to docs preview:
Conditional-
ROSA:
https://70909--ocpdocs-pr.netlify.app/openshift-rosa/latest/storage/understanding-persistent-storage#types-of-persistent-volumes_understanding-persistent-storage
https://70909--ocpdocs-pr.netlify.app/openshift-rosa/latest/storage/container_storage_interface/persistent-storage-csi#csi-drivers-supported_persistent-storage-csi
ENTERPRISE:
https://70909--ocpdocs-pr.netlify.app/openshift-enterprise/latest/storage/understanding-persistent-storage#types-of-persistent-volumes_understanding-persistent-storage
https://70909--ocpdocs-pr.netlify.app/openshift-enterprise/latest/storage/container_storage_interface/persistent-storage-csi#csi-drivers-supported_persistent-storage-csi
QE review:
Additional information: