Bug 1590967

What should we do in case same VDO volume is used for several storage domains?

What information should we provide in case of (multiple) vdo-thinp layers?

(In reply to Denis Chaplygin from comment #1)
> What should we do in case same VDO volume is used for several storage
> domains?
> 

I think the same space saving can be reported.

> What information should we provide in case of (multiple) vdo-thinp layers?

Adding Dennis for inputs.

Created attachment 1452039 [details]
RHHI Cockpit VDO Savings Picture

Savings for 2.0
Cockpit will display the VDO savings per node. See attachment.

Savings for 2.0+
Storage Domain

Each storage domain should display the storage savings by averaging each
VDO's volume savings on each gluster's volume bricks.

example:
the brick for vmstore on host-rhhi1 (/dev/sdb)
has space savings of 49% (vdo command: vdostats --hum)

the brick for vmstore on host-rhhi2 (/dev/sdb)
has space savings of 57%

the brick for vmstore on host-rhhi3 (/dev/sdb)
has space savings of 87%

(49+57+87)/3 = 64% space savings

64% space savings should be displayed for the storage domain after "Guaranteed Free Space" and before "Description" 
--------------------------------

Savings for 2.0+
Dashboard

The Dashboard could show the savings either in the storage "circle" as part of the ring or just as text in the center of the 
ring.  I would normally say that the savings would be green, but that is used already, so blue might work.  The savings 
here should report the savings for the cluster not just per storage domain.  

The stats can come from either VDO manager by using "vdostats --verbose" 
Command: vdostats --verbose|egrep "logical blocks used|physical blocks used"

  data blocks used                    : 17275885
  logical blocks used                 : 33712693

Or from sysfs

Sysfs does not hold a percent savings, which means you have to calculate it yourself.
 
1. capture the output of /sys/kvdo/<vdo volume name>/statistics/{logical_blocks_used|physical_blocks_used}
2. add the physical blocks used for all volumes in the cluster together
3. add the logical blocks used for all volumes in the cluster together 
4. subtract the total logical blocks used by total physical blocks used (this is the total number of blocks saved by VDO)
5. then divide the total saved blocks by the total logical blocks, multiplied by 100 to get the savings percent


ls /sys/kvdo/vdo_sdb/statistics/|egrep "logical blocks used|physical blocks used"
logical_blocks_used
data_blocks_used

cat $(ls -d /sys/kvdo/*/statistics/*|egrep "logical_blocks_used|data_blocks_used")
33712680
17275871

Math
33712680 - 17275871 = 16436059  (saved blocks)
16436059 / 33712680 = 0.4875 *100 = 48% (savings percentage)
0.4875 *100 = 48%

Is there a separate bug tracking the vdsm changes?

No

(In reply to Denis Chaplygin from comment #6)
> No

Can this bz be in modified without the vdsm patches - either add those patches here, or have another bug tracking vdsm changes which this bug is dependent on

It will not work without VDSM patches

(In reply to Denis Chaplygin from comment #8)
> It will not work without VDSM patches

Denis, please move bug state in that case, and add the required patches here

This bug has not been marked as blocker for 4.2.6 and we are now in blockers only phase. Please consider re-targeting this bug to next release or block the 4.2.6 release for this.

Moving back to POST having 2 referenced patches still unmerged.

Space savings are displayed on bricks

Advanced brick details always displays the savings% as 0
"Deduplication/Compression savings (%) as 0"

VDO space savings reported from CLI

On Node1
---------
[root@ ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdb      18.2T     41.2G     18.1T   0%           73%

On Node2
---------
[root@ ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdb      18.2T     41.2G     18.1T   0%           73%

On Node3
---------
[root@ ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdb      18.2T     41.1G     18.1T   0%           68%

(In reply to SATHEESARAN from comment #14)
> VDO space savings reported from CLI
> 
> On Node1
> ---------
> [root@ ~]# vdostats --human-readable
> Device                    Size      Used Available Use% Space saving%
> /dev/mapper/vdo_sdb      18.2T     41.2G     18.1T   0%           73%
> 
> On Node2
> ---------
> [root@ ~]# vdostats --human-readable
> Device                    Size      Used Available Use% Space saving%
> /dev/mapper/vdo_sdb      18.2T     41.2G     18.1T   0%           73%
> 
> On Node3
> ---------
> [root@ ~]# vdostats --human-readable
> Device                    Size      Used Available Use% Space saving%
> /dev/mapper/vdo_sdb      18.2T     41.1G     18.1T   0%           68%

can you attach the vdsm.log/supervdsm.log from one of the nodes?

Created attachment 1480826 [details]
vdo-savings

This is from a 4.2.5 deployment, so logs will help to analyze why the savings are not displayed

Tested this bug and could see the "Deduplication/Compression savings (%) as 0" though the cli "savings percent" was 50.
The savings % in CLI was varying while writing data to the volume but it wasn't reflecting in the UI. Attaching the relevant logs for further debugging.

[root@rhsqa-grafton7 ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdb      20.0T     31.9G     20.0T   0%           62%

[root@rhsqa-grafton7 ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdb      20.0T     39.3G     20.0T   0%           59%


[root@rhsqa-grafton7 ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdb      20.0T   1019.4G     19.0T   4%           50%


[root@rhsqa-grafton7 ~]# vdo status | grep "saving percent"
saving percent: 50

Created attachment 1486267 [details]
VDO_savings_Screenshot

Created attachment 1486268 [details]
vdsm1.log

Created attachment 1486269 [details]
Supervdsm.log

Created attachment 1486270 [details]
vdsm2.log

Created attachment 1486271 [details]
Engine.log

Tested the bug on rhvm-4.2.7.4. The fix seems to be working as expected.
The space savings is reflecting in the RHV-M corresponding to the CLI output.


[root@rhsqa-abc ~]# vdostats --human-readable
Device                    Size      Used Available Use% Space saving%
/dev/mapper/vdo_sdc     223.1G     60.0G    163.0G  26%           62%
/dev/mapper/vdo_sdd     931.0G      4.2G    926.8G   0%           77%


Attaching the screenshot of the RHV-M

Created attachment 1496372 [details]
Verified_UI_Screenshot_Space_Savings

Canceling the needinfo on the assignee,since no longer required.

This bugzilla is included in oVirt 4.2.7 release, published on November 2nd 2018.

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