Bug 1459894

Summary: Newly created LV for brick entry should be of type thinlv
Product: [oVirt] cockpit-ovirt Reporter: SATHEESARAN <sasundar>
Component: GdeployAssignee: Devyani Kota <dkota>
Status: CLOSED CURRENTRELEASE QA Contact: RamaKasturi <knarra>
Severity: medium Docs Contact:
Priority: medium    
Version: 0.10.7-0.0.18CC: bugs, knarra, lveyde, rbarry, sabose
Target Milestone: ovirt-4.1.5Flags: sasundar: ovirt-4.1?
sasundar: planning_ack?
rule-engine: devel_ack+
sasundar: testing_ack+
Target Release: 0.10.7-0.0.22   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: cockpit-ovirt-0.10.7-0.0.22 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-08-23 08:02:44 UTC Type: Bug
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: 1411323, 1485863    
Attachments:
Description Flags
Screenshot that shows that new LV entry without thinp checked
none
Attaching screenshot for thinp checkbox checked none

Description SATHEESARAN 2017-06-08 13:16:15 UTC
Description of problem:
-----------------------
While initiating 'Hosted engine with Gluster' from cockpit UI, then selecting the 'Bricks' tab of the deployment wizard and creating the new LV for bricks creates the LV (brick) entry with 'thinp' not checked by default, which means the LV is created thick. But its the thinlv that is suited/required for the RHHI deployment

Version-Release number of selected component (if applicable):
-------------------------------------------------------------
cockpit-ovirt-dashboard-0.10.7-0.0.18

How reproducible:
-----------------
Always

Steps to Reproduce:
-------------------
1. Start 'Hosted Engine with Gluster' deployment
2. On 'Bricks' tab, create one more LV for brick

Actual results:
---------------
The new LV entry doesn't have the thinp checkbox checked

Expected results:
------------------
The new LV entry should have the thinp checkbox checked (i.e) the newly created LV for the brick should be of type 'thin LV'

Comment 1 SATHEESARAN 2017-06-08 13:17:35 UTC
Created attachment 1286149 [details]
Screenshot that shows that new LV entry without thinp checked

Comment 2 RamaKasturi 2017-08-16 13:41:22 UTC
Verified and works fine with build cockpit-ovirt-dashboard-0.10.7-0.0.23.el7ev.noarch.

The new LV entry has the thinp checkbox checked. Attaching screenshot for the same.

Comment 3 RamaKasturi 2017-08-16 13:41:53 UTC
Created attachment 1314142 [details]
Attaching screenshot for thinp checkbox checked