Bug 1142762 (ovirt_refactor_disk_class_hierarchy) - [RFE][Tracker][CodeChange] Refactor Disks' class hierarchy
Summary: [RFE][Tracker][CodeChange] Refactor Disks' class hierarchy
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: ovirt_refactor_disk_class_hierarchy
Product: ovirt-engine
Classification: oVirt
Component: RFEs
Version: ---
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-4.0.0-rc
: 4.0.0
Assignee: Tal Nisan
QA Contact: Aharon Canan
URL:
Whiteboard:
Depends On:
Blocks: 920211 960169 1080097 1097843 1100999 1101011 1128724 1139879 1150239 1159640 1168572 1170577 1186817 1200736 1200741 1291177 1302752 ovirt_report_storage_iops_accum_per_vm 1317438 oVirt_turn_off_autoresume_of_paused_VMs
TreeView+ depends on / blocked
 
Reported: 2014-09-17 11:19 UTC by Allon Mureinik
Modified: 2017-02-12 13:44 UTC (History)
14 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-07-05 07:53:35 UTC
oVirt Team: Storage
rule-engine: ovirt-4.0.0+
acanan: testing_plan_complete-
ylavi: planning_ack+
amureini: devel_ack+
acanan: testing_ack+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1139879 1 None None None 2021-01-20 06:05:38 UTC
Red Hat Bugzilla 1158463 0 low CLOSED [RFE] [CodeChange] Refactor storage helpers in oVirt Engine 2021-02-22 00:41:40 UTC
oVirt gerrit 38650 0 master NEW core: Moved storage related entities to the storage package Never
oVirt gerrit 38652 0 None NEW core: Extract DiskStorageType from Disk entity to a standalone class Never
oVirt gerrit 38659 0 master NEW core: Renamed storage entities as per Java naming standards Never
oVirt gerrit 56691 0 master ABANDONED core: Introduce boot field in the DiskVmElement entity 2016-05-18 14:15:07 UTC
oVirt gerrit 56692 0 master ABANDONED core: Remove boot property from the BaseDisk entity 2016-05-18 14:17:02 UTC
oVirt gerrit 57603 0 master MERGED core: Move disks boot and interface properties to DiskVmelement 2016-05-18 20:06:10 UTC

Internal Links: 1139879 1158463

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