Bug 1686258 - Creation of multiple bricks,one of them being arbiter, on the same disk without dedupe & compression, results in failure
Summary: Creation of multiple bricks,one of them being arbiter, on the same disk witho...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat Storage
Component: rhhi
Version: rhhiv-1.6
Hardware: x86_64
OS: Linux
high
medium
Target Milestone: ---
: RHHI-V 1.6
Assignee: Sahina Bose
QA Contact: SATHEESARAN
URL:
Whiteboard:
Depends On: 1686259
Blocks: RHHI-V-1-6-Engineering-InFlight-BZs
TreeView+ depends on / blocked
 
Reported: 2019-03-07 06:04 UTC by Mugdha Soni
Modified: 2019-05-09 06:09 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1686259 (view as bug list)
Environment:
Last Closed: 2019-05-09 06:09:09 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2019:1121 0 None None None 2019-05-09 06:09:18 UTC

Description Mugdha Soni 2019-03-07 06:04:51 UTC
Description of problem:
------------------------
When multiple thinp volumes are carved out of same disk then in inventory file there is presence of multiple thinpools.

Version-Release number of selected component :
-----------------------------------------------
rhvh-4.3.0.5-0.20190305
gluster-ansible-maintenance-1.0.1-1.el7rhgs.noarch
gluster-ansible-roles-1.0.4-4.el7rhgs.noarch
gluster-ansible-infra-1.0.3-3.el7rhgs.noarch
gluster-ansible-repositories-1.0-1.el7rhgs.noarch
gluster-ansible-features-1.0.4-5.el7rhgs.noarch
gluster-ansible-cluster-1.0-1.el7rhgs.noarch


How reproducible:
-------------------
Everytime


Steps to Reproduce:
--------------------
1.In gluster deployment create multiple thinp volumes on the same disk.


Actual results:
-------------------
Multiple thinpools are present in inventory file

Expected results:
---------------------
Single thinpool section should be present under gluster_infra_thinpools if thinp volumes are carved out of same disk.

Additional info:
--------------------
Here is the snippet of the gluster infra thinpool section.

 gluster_infra_thick_lvs:
        - vgname: gluster_vg_sdb
          lvname: gluster_lv_engine
          size: 100G
        - vgname: gluster_vg_sdc
          lvname: gluster_lv_data
          size: 4000G
        - vgname: gluster_vg_sdc
          lvname: gluster_lv_vmstore
          size: 2000G
      gluster_infra_thinpools:
        - vgname: gluster_vg_sdd
          thinpoolname: gluster_thinpool_gluster_vg_sdd
          poolmetadatasize: 3
        - vgname: gluster_vg_sdd
          thinpoolname: gluster_thinpool_gluster_vg_sdd
          poolmetadatasize: 3

Comment 1 SATHEESARAN 2019-03-07 17:17:16 UTC
This issue is less severe, because that the ansible preserves the idempotent behaviour, there is
no attempt to create the thinpool once again.

Yet, its safe to make the required correction.

Moving this bug to RHHI-1.7

Comment 2 SATHEESARAN 2019-03-20 08:00:04 UTC
(In reply to SATHEESARAN from comment #1)
> This issue is less severe, because that the ansible preserves the idempotent
> behaviour, there is
> no attempt to create the thinpool once again.
> 
> Yet, its safe to make the required correction.
> 
> Moving this bug to RHHI-1.7

My initial estimation was incorrect. While regression, ran in to a test scenario,
1. No dedupe & compression on the brick. So brick is a thin LV
2. Create multiple bricks on the same disk
3. One of them is arbiter.

In this scenario, the gluster ansible playbook ends up like this:
<snip>
gluster_infra_thinpools:
        - vgname: gluster_vg_sdc
          thinpoolname: gluster_thinpool_gluster_vg_sdc
          poolmetadatasize: 1
        - vgname: gluster_vg_sdc
          thinpoolname: gluster_thinpool_gluster_vg_sdc
          poolmetadatasize: 16

</snip>

So as per the original intent behind the bug, for each brick on the same disk, 
creation of dedicated thinpool is attempted. In this case, thinpool 'gluster_thinpool_gluster_vg_sdc'
is created with poolmetadatasize of 1, which looks too small for the other thin LV
and LV complains about it and fails.

This bug is now considered as a blocker from QE perspective

Comment 5 SATHEESARAN 2019-03-29 18:49:53 UTC
Tested with cockpit-ovirt-dashboard-0.12.6

When there are multiple thin LVs are created on the same disk, 
there only one thinpool created.

Snip from generated ansible vars file
--------------------------------------
    gluster_infra_thinpools:
        - vgname: gluster_vg_sdc
          thinpoolname: gluster_thinpool_gluster_vg_sdc
          poolmetadatasize: 5G
      gluster_infra_lv_logicalvols:
        - vgname: gluster_vg_sdc
          thinpool: gluster_thinpool_gluster_vg_sdc
          lvname: gluster_lv_data
          lvsize: 300G
        - vgname: gluster_vg_sdc
          thinpool: gluster_thinpool_gluster_vg_sdc
          lvname: gluster_lv_vmstore
          lvsize: 600G
        - vgname: gluster_vg_sdc
          thinpool: gluster_thinpool_gluster_vg_sdc
          lvname: gluster_lv_newvol
          lvsize: 900G

Comment 7 errata-xmlrpc 2019-05-09 06:09:09 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

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

https://access.redhat.com/errata/RHEA-2019:1121


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