Bug 1980340 - [DOC-OCPRHV] Add resize to ovirt CSI driver
Summary: [DOC-OCPRHV] Add resize to ovirt CSI driver
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Storage
Version: 4.8
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ---
: 4.9.0
Assignee: Steve Goodman
QA Contact: michal
URL:
Whiteboard:
Depends On: 1953674
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-07-08 12:25 UTC by ctomasko
Modified: 2021-08-09 15:23 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of: 1953674
Environment:
Last Closed: 2021-08-09 15:20:06 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 2 Steve Goodman 2021-07-13 07:39:04 UTC
1. CSI drivers supported by OpenShift container platform [1] Table 1:

   Add a checkmark for CSI resize for RHV.

2. oVirt CSI driver storage class [2] (code example):
   <1> as-is
   <2> Change "false" to "<boolean>" with explanation as-is.
   <3> as-is
   <4> Change "true" to "<boolean>" with explanation: if "true" thin provisoning, if "false" preallocated. Thin provisioning is recommended. (Gal needs to check this out.
   <5> possible values: ext4 or xfs
3. Creating a persistent volume on RHV > pvc-ovirt.yaml:
----
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: pvc-ovirt
spec:
  storageClassName: ovirt-csi-sc 
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
     storage: <volume size>  (Add one space to indent it as shown here).
  volumeMode: <volume mode> 
----


4. Code example under "oVirt CSI driver storage class" [4]

Add new stuff:
----
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name:  <storage-class-name>
  annotations:
    storageclass.kubernetes.io/is-default-class: <bool>
provisioner: csi.ovirt.org
allowVolumeExpansion: <bool> ** recommended true
reclaimPolicy: Delete
volumeBindingMode: Immediate
parameters:
  storageDomainName: <rhv-storage-domain-name>
  thinProvisioning: <bool> ** recommended true
  csi.storage.k8s.io/fstype: <ext4|xfs>
----

5. Add a section for RHV/oVirt just as there are specifics for other providers. Use code example from previous item.


[1] https://docs.openshift.com/container-platform/4.7/storage/container_storage_interface/persistent-storage-csi.html#csi-drivers-supported_persistent-storage-csi

[2] https://docs.openshift.com/container-platform/4.7/storage/container_storage_interface/persistent-storage-csi-ovirt.html#ovirt-csi-driver-storage-class_persistent-storage-csi-ovirt

[3] https://docs.openshift.com/container-platform/4.7/storage/container_storage_interface/persistent-storage-csi-ovirt.html#persistent-storage-rhv_persistent-storage-csi-ovirt

[4] https://docs.openshift.com/container-platform/4.7/storage/container_storage_interface/persistent-storage-csi-ovirt.html#ovirt-csi-driver-storage-class_persistent-storage-csi-ovirt

[5] https://docs.openshift.com/container-platform/4.7/post_installation_configuration/storage-configuration.html#defining-storage-classes_post-install-storage-configuration

Comment 3 Steve Goodman 2021-07-13 11:53:23 UTC
I created a PR for this: https://github.com/openshift/openshift-docs/pull/34475

Gal, please review.

Comment 5 Gal Zaidman 2021-07-14 07:51:06 UTC
(In reply to Steve Goodman from comment #3)
> I created a PR for this:
> https://github.com/openshift/openshift-docs/pull/34475
> 
> Gal, please review.

Done

Comment 6 Steve Goodman 2021-07-15 09:49:44 UTC
(In reply to Gal Zaidman from comment #5)
 
> Done

I implemented your comments, please approve.

Comment 7 Steve Goodman 2021-07-18 05:05:21 UTC
Michal,

Could you please take a look at this? Links to the preview are included in the PR description.

Comment 8 Steve Goodman 2021-08-05 09:13:47 UTC
Michal,

Can you please mark this as VERIFIED or let me know what else needs to be fixed?


Note You need to log in before you can comment on or make changes to this bug.