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

Bug 1738019

Summary: Update LV cache related variables in generated vars file
Product: [oVirt] cockpit-ovirt Reporter: Gobinda Das <godas>
Component: gluster-ansibleAssignee: Gobinda Das <godas>
Status: CLOSED CURRENTRELEASE QA Contact: SATHEESARAN <sasundar>
Severity: high Docs Contact:
Priority: unspecified    
Version: 0.13.1CC: bugs, godas, rhs-bugs, sabose, sasundar
Target Milestone: ovirt-4.3.6Keywords: ZStream
Target Release: 0.13.6Flags: sbonazzo: ovirt-4.3?
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: cockpit-ovirt-0.13.6 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1734717 Environment:
Last Closed: 2019-09-26 19:43:19 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Gluster RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1734717    

Description Gobinda Das 2019-08-06 12:32:10 UTC
Description of problem:
-------------------------
Few of the params in lvcache section are not required and will be auto-computed

Following variables are not required:
          cachethinpoolname: gluster_thinpool_gluster_vg_sdd
          cachemetalvsize: 2G


Version-Release number of selected component (if applicable):
---------------------------------------------------------------
RHHI-V 1.6
cockpit-ovirt-dashboard-0.13.5


How reproducible:
-----------------
NA


Steps to Reproduce:
--------------------
1. Enable lv cache 

Actual results:
---------------
1. cache lv meta size is computed
2. lv cache meta name is computed

Expected results:
------------------
When the above said options are not mentioned, lv cache meta is automatically computed by LV which is more safer


Additional info:

Comment 2 SATHEESARAN 2019-08-06 15:17:07 UTC
(In reply to Gobinda Das from comment #0)
> Description of problem:
> -------------------------
> Few of the params in lvcache section are not required and will be
> auto-computed
> 
> Following variables are not required:
>           cachethinpoolname: gluster_thinpool_gluster_vg_sdd
>           cachemetalvsize: 2G
Incorrect description.
It was not the **cachethinpoolname** but the **cachemetalvname**

Comment 3 SATHEESARAN 2019-09-03 05:16:48 UTC
Tested with cockpit-ovirt-dashboard-0.13.6

All the optional params for lvcache are removed, leaving LVM
to compute it by default.

LV cache related params in the generated vars file looks like:

<snip>
      gluster_infra_cache_vars:
        - vgname: gluster_vg_sdb
          cachedisk: '/dev/sdb,/dev/sdc'
          cachelvname: cachelv_gluster_thinpool_gluster_vg_sdb
          cachethinpoolname: gluster_thinpool_gluster_vg_sdb
          cachelvsize: 225G
          cachemode: writeback
</snip>

Comment 4 Sandro Bonazzola 2019-09-26 19:43:19 UTC
This bugzilla is included in oVirt 4.3.6 release, published on September 26th 2019.

Since the problem described in this bug report should be
resolved in oVirt 4.3.6 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.