Bug 1629543 - The LV size should be equal to the logical size on VDO enabled volumes
Summary: The LV size should be equal to the logical size on VDO enabled volumes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: cockpit-ovirt
Classification: oVirt
Component: Gdeploy
Version: 0.11.33
Hardware: x86_64
OS: Linux
high
high
Target Milestone: ovirt-4.3.2
: 0.12.5
Assignee: Gobinda Das
QA Contact: bipin
URL:
Whiteboard:
Depends On:
Blocks: 1629542
TreeView+ depends on / blocked
 
Reported: 2018-09-17 04:22 UTC by bipin
Modified: 2019-03-19 10:03 UTC (History)
9 users (show)

Fixed In Version: cockpit-ovirt-0.12.5
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1629542
Environment:
Last Closed: 2019-03-19 10:03:20 UTC
oVirt Team: Gluster
Embargoed:
rule-engine: ovirt-4.3+


Attachments (Terms of Use)
UI_Screenshot (21.04 KB, image/png)
2018-09-17 04:22 UTC, bipin
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 94671 0 master MERGED Adding Info icon next to LV Size and Dedupe and Compression Check box. This icon will only be visible when VDO is enable... 2019-01-08 08:57:57 UTC
oVirt gerrit 96655 0 ovirt-4.2 ABANDONED Adding Info icon next to LV Size and Dedupe and Compression Check box. This icon will only be visible when VDO is enable... 2019-02-21 07:38:26 UTC
oVirt gerrit 97944 0 master MERGED The LV size and logical size are same on VDO enabled volumes. 2019-03-11 13:02:38 UTC
oVirt gerrit 98445 0 ovirt-4.3 MERGED The LV size and logical size are same on VDO enabled volumes. 2019-03-11 13:32:47 UTC

Description bipin 2018-09-17 04:22:22 UTC
Created attachment 1483855 [details]
UI_Screenshot

+++ This bug was initially created as a clone of Bug #1629542 +++

Description of problem:
=======================
In currently implementation,the LV size is always lesser than the logical size while creating the VDO volumes. Going forward the LV should be equal to the logical size to fully utilize the underline storage.


Version-Release number of selected component (if applicable):
============================================================
cockpit-ovirt-dashboard-0.11.33-1.el7ev.noarch

How reproducible:
=================
100%

Steps to Reproduce:
1.
2.
3.

Actual results:
==============
The LV size is lesser than logical size

Expected results:
=================
The LV size should be equal to logical size

Additional info:
===============
Adding the screenshot from the cockpit UI

Comment 1 Gobinda Das 2018-09-19 06:10:20 UTC
Parth please look this as priority.

Comment 2 Gobinda Das 2018-09-24 07:47:47 UTC
@Parth Provide info for LVM Size filed that the lvm will be created with given size i.e. input of the LVM will not be same with VDO logical size.

Comment 3 Parth Dhanjal 2018-09-24 10:33:31 UTC
(In reply to Gobinda Das from comment #2)
> @Parth Provide info for LVM Size filed that the lvm will be created with
> given size i.e. input of the LVM will not be same with VDO logical size.

Hey Gobinda! I'll add an Info Icon right next to the LVM Cache Size field.

Comment 4 Sahina Bose 2018-10-10 12:10:47 UTC
Are we only fixing this via info icon - I thought we had discussed that LV size would change to vdo logical size?

Comment 5 Gobinda Das 2018-10-10 13:03:51 UTC
sahina, Now vdo logical size calculated based on lv size. It's required redesign of page to fix. So now we gave info to user via info icon.

Comment 6 Gobinda Das 2018-10-10 13:51:18 UTC
This is a known issue.

Comment 7 Sandro Bonazzola 2019-01-28 09:40:05 UTC
This bug has not been marked as blocker for oVirt 4.3.0.
Since we are releasing it tomorrow, January 29th, this bug has been re-targeted to 4.3.1.

Comment 8 Gobinda Das 2019-02-27 10:59:34 UTC
Ovirt-4.3.1 already released, so moving to ovirt-4.3.2

Comment 9 bipin 2019-03-19 07:32:55 UTC
Verified the bug on the latest cockpit dashboard and could see the LV size is equal to the logical size of VDO volume

Components version:
==================
cockpit-ovirt-dashboard-0.12.5-1.el7ev.noarch
rhvh-4.3.0.5-0.20190313.0+1

Steps:
=====
1. Check the VDO box for the brick you want to have dedupe and compression enabled
2. Once deployed compare the size of the LV and VDO logical size

Could see that the ansible playbook generated in the review tab had proper values set.

Output:
======
sdb                                                                8:16   0    20T  0 disk 
└─vdo_sdb                                                        253:12   0    80T  0 vdo  
  ├─gluster_vg_sdb-gluster_lv_data                               253:14   0    40T  0 lvm  /gluster_bricks/data
  └─gluster_vg_sdb-gluster_lv_vmstore                            253:15   0    40T  0 lvm  /gluster_bricks/vmstore


[root@rhsqa-abc log]# lvs
  LV                          VG                       Attr       LSize    Pool   Origin                    Data%  Meta%  Move Log Cpy%Sync Convert
  gluster_lv_data             gluster_vg_sdb           -wi-ao----   40.00t                                                                         
  gluster_lv_vmstore          gluster_vg_sdb           -wi-ao----  <40.00t                                                                         


Moving to verified based on above result

Comment 10 Sandro Bonazzola 2019-03-19 10:03:20 UTC
This bugzilla is included in oVirt 4.3.2 release, published on March 19th 2019.

Since the problem described in this bug report should be
resolved in oVirt 4.3.2 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.


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