Bug 1610703

Summary: Cockpit Day 2: Thinp checkbox under brick tab should be read only as in Day 1 operation
Product: [oVirt] cockpit-ovirt Reporter: bipin <bshetty>
Component: GdeployAssignee: Parth Dhanjal <dparth>
Status: CLOSED CURRENTRELEASE QA Contact: SATHEESARAN <sasundar>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 0.11.7CC: bugs, godas, rhs-bugs, sabose, sankarshan, sasundar
Target Milestone: ovirt-4.3.2Flags: rule-engine: ovirt-4.3?
bshetty: planning_ack?
rule-engine: devel_ack+
sasundar: testing_ack+
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: cockpit-ovirt-0.11.38-1.el7ev Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1610699 Environment:
Last Closed: 2019-03-19 10:05:21 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: 1610699    
Attachments:
Description Flags
Screenshot1
none
Screenshot2
none
Screenshot of bricks tab none

Description bipin 2018-08-01 08:56:41 UTC 
+++ This bug was initially created as a clone of Bug #1610699 +++

Description of problem:
======================
Currently in Day 1, the thinp checkbox is made ready only and gets automatically unchecked while checking enable dedupe and compression. Same should be implemented in Day 2 ,which is currently read/write.


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

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

Steps to Reproduce:
==================
1.Login to cockpit -> Manage gluster -> Create volume
2.Navigate to the bricks tab

Actual results:
==============
Both thinp and enable dedupe & compression can be checked for a same device


Expected results:
================
Should make thinp checkbox readonly 


Additional info:

Attaching the screenshot for cockpit day 1 and day 2

--- Additional comment from bipin on 2018-08-01 04:53 EDT ---



--- Additional comment from bipin on 2018-08-01 04:54 EDT ---
Comment 1 bipin 2018-08-01 08:57:22 UTC 
Created attachment 1472044 [details]
Screenshot1
Comment 2 bipin 2018-08-01 08:57:49 UTC 
Created attachment 1472045 [details]
Screenshot2
Comment 3 Mugdha Soni 2019-01-17 10:42:57 UTC 
Created attachment 1521254 [details]
Screenshot of bricks tab
Comment 4 Mugdha Soni 2019-01-17 10:43:15 UTC 
Moving the bug to assigned state as when dedeupe and compression checkbox is checked the thinp checkbox for the first brick is not unchecked .This is possible in both the scenario when bricks are carved out of same disk or different disk.
Comment 5 SATHEESARAN 2019-01-17 11:34:26 UTC 
Moving the target_milestone to 4.2.9
Comment 6 Sandro Bonazzola 2019-01-18 15:38:31 UTC 
Resetting the target milestone to 4.3.0, there won't be any 4.2.9 after 4.3.0 GA
Comment 7 Sandro Bonazzola 2019-01-28 09:39:53 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:54:32 UTC 
Ovirt-4.3.1 already release, so moving to ovirt-4.3.2
Comment 9 SATHEESARAN 2019-03-12 17:15:19 UTC 
Tested with cockpit-ovirt-dashboard-0.12.4

1. thinp checkbox is made read-only
Comment 10 Sandro Bonazzola 2019-03-19 10:05:21 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.