Bug 833076

Summary: [Storage] GUI should filter shared disks list and not show disk that already connected on VM it conneted to.
Product: Red Hat Enterprise Virtualization Manager Reporter: Leonid Natapov <lnatapov>
Component: ovirt-engineAssignee: Gilad Chaplik <gchaplik>
Status: CLOSED CURRENTRELEASE QA Contact: Leonid Natapov <lnatapov>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.1.0CC: abaron, acathrow, amureini, dfediuck, dyasny, ecohen, hateya, iheim, lpeer, Rhev-m-bugs, yeylon, ykaul
Target Milestone: ---   
Target Release: 3.1.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: storage
Fixed In Version: SI9 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 20:01:49 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:

Description Leonid Natapov 2012-06-18 14:24:54 UTC 
[Storage] GUI should filter shared disks list and not show disk that already connected on VM it conneted to.

How to repoduce:
1.Create VM (VM1)
2.Create shared disk.(disk1)
3.Attach shared disk (disk1) to this VM (VM1)

Now,stay on VM1 and click on attach again. You will see the disk you already attached. If you will try to attach it again nothing will happen in the UI and engine will through an exception. Another BZ will be opened for backend to handle the exception.
Comment 1 Gilad Chaplik 2012-06-25 13:31:29 UTC 
(In reply to comment #0)
> [Storage] GUI should filter shared disks list and not show disk that already
> connected on VM it conneted to.
> 
> How to repoduce:
> 1.Create VM (VM1)
> 2.Create shared disk.(disk1)
> 3.Attach shared disk (disk1) to this VM (VM1)
> 
> Now,stay on VM1 and click on attach again. You will see the disk you already
> attached. If you will try to attach it again nothing will happen in the UI
> and engine will through an exception. Another BZ will be opened for backend
> to handle the exception.

what is the other BZ?
Comment 2 Leonid Natapov 2012-06-25 13:39:25 UTC 
https://bugzilla.redhat.com/show_bug.cgi?id=833077
Comment 3 Gilad Chaplik 2012-06-25 14:30:47 UTC 
pushed upstream (http://gerrit.ovirt.org/#/c/5680/)
Comment 4 Gilad Chaplik 2012-06-26 07:50:21 UTC 
changed the backend query to filter out disks that are already attached to the VM,  changing component to backend.
Comment 5 Gilad Chaplik 2012-06-26 08:17:33 UTC 
merged upstream (http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=commit;h=3cc5d4086bd9617173eea38c17925b3196b87a00)
Comment 6 Leonid Natapov 2012-07-09 10:39:49 UTC 
SI9. fixed.