Bug 1142762 (ovirt_refactor_disk_class_hierarchy)

Summary:	[RFE][Tracker][CodeChange] Refactor Disks' class hierarchy
Product:	[oVirt] ovirt-engine	Reporter:	Allon Mureinik <amureini>
Component:	RFEs	Assignee:	Tal Nisan <tnisan>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Aharon Canan <acanan>
Severity:	high	Docs Contact:
Priority:	high
Version:	---	CC:	ahadas, amureini, bugs, gklein, iheim, jcall, michal.skrivanek, rbalakri, scohen, s.kieske, srevivo, tnisan, ykaul, ylavi
Target Milestone:	ovirt-4.0.0-rc	Keywords:	CodeChange, FutureFeature, Tracking
Target Release:	4.0.0	Flags:	rule-engine: ovirt-4.0.0+ acanan: testing_plan_complete- ylavi: planning_ack+ amureini: devel_ack+ acanan: testing_ack+
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-07-05 07:53:35 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	920211, 960169, 1080097, 1097843, 1100999, 1101011, 1128724, 1139879, 1150239, 1159640, 1168572, 1170577, 1186817, 1200736, 1200741, 1291177, 1302752, 1317433, 1317438, 1317450

Description Allon Mureinik 2014-09-17 11:19:08 UTC

Description of problem:
The Disks hierarchy classes currently represent three types of properties;
- Logic properties (e.g., alias)
- Storage properties (e.g., real size, format)
- VM connection properties (r/o, interface)

The classes should be refactored to represent this.
This refactoring will facilitate many features, especially the new storage backends (non-SPM oVirt storage, Ceph).

Comment 1 Allon Mureinik 2015-04-12 13:45:36 UTC

This is a must-have for critical 3.6.0 features, such as Cinder.

Comment 2 Allon Mureinik 2015-04-27 17:09:43 UTC

CodeChange required for 3.6.0, targeting back as per face-to-face discussion with Yaniv.

Comment 3 Red Hat Bugzilla Rules Engine 2015-10-19 10:55:06 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 4 Yaniv Lavi 2016-03-03 13:46:43 UTC

Is this done? All the patches attached are merged.

Comment 5 Allon Mureinik 2016-03-06 08:46:33 UTC

(In reply to Yaniv Dary from comment #4)
> Is this done?
No

> All the patches attached are merged.
True, but many other patches aren't even submitted yet.

Comment 8 Sandro Bonazzola 2016-05-02 10:00:43 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 9 Aharon Canan 2016-06-08 12:05:29 UTC

following my discussion with Tal no special tests needed here, only regression. 
Moving to verified.

Comment 10 Sandro Bonazzola 2016-07-05 07:53:35 UTC

oVirt 4.0.0 has been released, closing current release.