Bug 1211140

Summary:	Gluster: prevent gluster brick creation on mbr partitions
Product:	[oVirt] ovirt-engine	Reporter:	RamaKasturi <knarra>
Component:	Frontend.WebAdmin	Assignee:	Ramesh N <rnachimu>
Status:	CLOSED WONTFIX	QA Contact:	Lubos Trilety <ltrilety>
Severity:	low	Docs Contact:
Priority:	low
Version:	---	CC:	bugs, gklein, knarra, lsurette, ltrilety, mbukatov, mgoldboi, nlevinki, rbalakri, rhs-bugs, rhsc-qe-bugs, rnachimu, sabose, sankarshan, sgraf, shtripat, srevivo, ykaul
Target Milestone:	---	Flags:	sbonazzo: ovirt-4.1-
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1210841
Clones:	1230338 1286916 (view as bug list)		Environment:
Last Closed:	2016-12-22 06:46:17 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:	1230338, 1286916

Description RamaKasturi 2015-04-13 06:46:56 UTC

+++ This bug was initially created as a clone of Bug #1210841 +++

Description of problem:
Extended partition is shown as available for bricks creation, however when someone wants to create a brick on such partition it fails with unexpected exception.

Version-Release number of selected component (if applicable):
ovirt-engine-3.6.0-0.0

How reproducible:
100%

Steps to Reproduce:
1. Create some extended partition on host
2. Go to 'Storage Device' tab check availability of the partition and try to create a brick on it


Actual results:
Unexpected exception

Expected results:
The partition should not be available it should be shown with a lock or add some functionality which creates a proper logical partition first.

Additional info:

--- Additional comment from Lubos Trilety on 2015-04-10 12:59:07 EDT ---

Such partition stays to be shown as available even when it is completely used.

Comment 1 Sahina Bose 2015-04-20 07:05:20 UTC

Marking it as low priority as extended partition is not a common use case for gluster.

Comment 2 Red Hat Bugzilla Rules Engine 2015-10-19 11:02:59 UTC

Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 3 Martin Bukatovic 2015-10-19 15:56:49 UTC

To align with BZ 1270792, I would propose to disallow admin to create brick on
extended partition.

Comment 4 Sandro Bonazzola 2015-10-26 12:47:49 UTC

this is an automated message. oVirt 3.6.0 RC3 has been released and GA is targeted to next week, Nov 4th 2015.
Please review this bug and if not a blocker, please postpone to a later release.
All bugs not postponed on GA release will be automatically re-targeted to

- 3.6.1 if severity >= high
- 4.0 if severity < high

Comment 5 Sandro Bonazzola 2015-11-05 08:20:30 UTC

oVirt 3.6.0 has been released on November 4th, re-targeting to 4.0 since this bug has been marked with severity < high

Comment 6 Red Hat Bugzilla Rules Engine 2015-11-16 14:07:45 UTC

This bug is flagged for 3.6, yet the milestone is for 4.0 version, therefore the milestone has been reset.
Please set the correct milestone or add the flag.

Comment 7 Ramesh N 2015-12-01 05:47:17 UTC

(In reply to Martin Bukatovic from comment #3)
> To align with BZ 1270792, I would propose to disallow admin to create brick
> on
> extended partition.

Brick creation should be blocked on any partitions not just extended partition. Changing the bug description accordingly.

Comment 8 Red Hat Bugzilla Rules Engine 2015-12-11 02:16:34 UTC

This bug is not marked for z-stream, yet the milestone is for a z-stream version, therefore the milestone has been reset.
Please set the correct milestone or add the z-stream flag.

Comment 9 Sandro Bonazzola 2016-05-02 10:05:09 UTC

Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 10 Yaniv Lavi 2016-05-23 13:19:50 UTC

oVirt 4.0 beta has been released, moving to RC milestone.

Comment 11 Yaniv Lavi 2016-05-23 13:23:31 UTC

oVirt 4.0 beta has been released, moving to RC milestone.

Comment 12 Ramesh N 2016-06-15 08:58:16 UTC

Considering this is not a regular scenario to create a MBR partition and trying to create brick on top of it, I would propose to close this bug as won't fix.

Kasturi, please let me know if you have any objects for closing this bug as won't fix.

Comment 13 Martin Bukatovic 2016-06-15 09:10:47 UTC

(In reply to Ramesh N from comment #12)
> Considering this is not a regular scenario to create a MBR partition and
> trying to create brick on top of it, I would propose to close this bug as
> won't fix.

Exactly, this is not a valid scenario which would break data alignment and
for that reason, it would make sense to prevent admin to configure bricks
in this way by mistake.

Comment 14 RamaKasturi 2016-06-15 09:17:36 UTC

yes, you can close this bug.

Comment 15 Stanislav Graf 2016-06-15 09:20:05 UTC

(In reply to Martin Bukatovic from comment #13)
> (In reply to Ramesh N from comment #12)
> > Considering this is not a regular scenario to create a MBR partition and
> > trying to create brick on top of it, I would propose to close this bug as
> > won't fix.
> 
> Exactly, this is not a valid scenario which would break data alignment and
> for that reason, it would make sense to prevent admin to configure bricks
> in this way by mistake.

Exactly, +1

Comment 16 Sahina Bose 2016-12-22 06:46:17 UTC

Low priority, closing