Bug 1159640 - [RFE] - remove Interface from add floating disk dialog
Summary: [RFE] - remove Interface from add floating disk dialog
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: RFEs
Version: 3.5.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.0.6
: ---
Assignee: Tal Nisan
QA Contact: Raz Tamir
URL:
Whiteboard:
Depends On: ovirt_refactor_disk_class_hierarchy
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-11-02 13:52 UTC by Ori Gofen
Modified: 2017-01-31 13:14 UTC (History)
12 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-01-31 13:14:54 UTC
oVirt Team: Storage
Embargoed:
ylavi: ovirt-future?
rule-engine: planning_ack?
rule-engine: devel_ack?
rule-engine: testing_ack?

Attachments (Terms of Use)
image (48.79 KB, image/png)
2014-11-02 13:52 UTC, Ori Gofen 		no flags Details
View All

Description Ori Gofen 2014-11-02 13:52:04 UTC 
Created attachment 952907 [details]
image

Description of problem:

Creating sPAPR-VSCSi image shouldn't be allowed on non-PowerKVM cluster,
The created image takes space, while can't be plugged to VM's (see image)

Version-Release number of selected component (if applicable):
vt8

How reproducible:
100%

Actual results:

Creating sPAPR-VSCSi image is allowed Via disks Tab

Expected results:
No sPAPR-VSCSi entry should be on non-PowerKVM cluster

Additional info:
Comment 1 Daniel Erez 2014-11-04 11:55:51 UTC 
Disk interface should be an attribute of VM-Disk attachment. Hence, should be removed from 'add floating disk' dialog (as it's not relevant as a property of a disk). This change would be covered as part of bug 1142762.
Comment 2 Allon Mureinik 2015-04-12 13:43:33 UTC 
This is part of bug 1142762 - must have for 3.6.0.
Comment 4 Yaniv Lavi 2016-12-05 14:12:53 UTC 
Was this done in the refactoring process?
Comment 5 Allon Mureinik 2016-12-06 12:29:57 UTC 
(In reply to Yaniv Dary from comment #4)
> Was this done in the refactoring process?
As far as I can see (both by opening the GUI and by examining the code), the answer is yes.

Tal - please keep me honest here.
If I'm right, please close this as CURRENT_RELEASE.
If not, please advise what's still missing.
Note You need to log in before you can comment on or make changes to this bug.