Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1980340

Summary: [DOC-OCPRHV] Add resize to ovirt CSI driver
Product: OpenShift Container Platform Reporter: ctomasko <ctomasko>
Component: StorageAssignee: Steve Goodman <sgoodman>
Storage sub component: oVirt CSI Driver QA Contact: michal <mgold>
Status: CLOSED NEXTRELEASE Docs Contact:
Severity: medium    
Priority: high CC: gzaidman, lleistne, mburman, mgold
Version: 4.8Keywords: Documentation
Target Milestone: ---   
Target Release: 4.9.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: 1953674 Environment:
Last Closed: 2021-08-09 15:20:06 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1953674    
Bug Blocks:    

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?