Bug 1610703 - Cockpit Day 2: Thinp checkbox under brick tab should be read only as in Day 1 operation
Summary: Cockpit Day 2: Thinp checkbox under brick tab should be read only as in Day 1...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: cockpit-ovirt
Classification: oVirt
Component: Gdeploy
Version: 0.11.7
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ovirt-4.3.2
: ---
Assignee: Parth Dhanjal
QA Contact: SATHEESARAN
URL:
Whiteboard:
Depends On:
Blocks: 1610699
TreeView+ depends on / blocked
 
Reported: 2018-08-01 08:56 UTC by bipin
Modified: 2019-03-19 10:05 UTC (History)
6 users (show)

Fixed In Version: cockpit-ovirt-0.11.38-1.el7ev
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1610699
Environment:
Last Closed: 2019-03-19 10:05:21 UTC
oVirt Team: Gluster
Embargoed:
rule-engine: ovirt-4.3?
bshetty: planning_ack?
rule-engine: devel_ack+
sasundar: testing_ack+

Attachments (Terms of Use)
Screenshot1 (70.58 KB, image/png)
2018-08-01 08:57 UTC, bipin 		no flags Details
Screenshot2 (47.46 KB, image/png)
2018-08-01 08:57 UTC, bipin 		no flags Details
Screenshot of bricks tab (78.80 KB, image/png)
2019-01-17 10:42 UTC, Mugdha Soni 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 94309 0 master MERGED Addressing 3 bugs. Bug#1 - The thinp option is read-only now. And by default it is checked. Bug#2 - The hostname field u... 2018-09-14 15:15:41 UTC
oVirt gerrit 94353 0 ovirt-4.2 MERGED Addressing 3 bugs. Bug#1 - The thinp option is read-only now. And by default it is checked. Bug#2 - The hostname field u... 2018-09-14 15:26:52 UTC
oVirt gerrit 97079 0 master MERGED Day 2: Thinp checkbox under brick tab should be read only and disabled when VDO is enabled. 2019-01-22 08:35:07 UTC

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.
Note You need to log in before you can comment on or make changes to this bug.