Tomas Jelinek has submitted this change and it was merged. Change subject: core: Added a possibility to filter disks according explicitely set values ...................................................................... core: Added a possibility to filter disks according explicitely set values Added new query GetFilteredAttachableDisksQuery to filter by explicitely set OS and VDS Group. Needed when showing the attach disk dialog from new VM dialog where the VM is not yet created but the OS and the cluster is already known. Feature Page: http://www.ovirt.org/Features/Instance_Types Change-Id: If363cd92aa0071c88f2333294002ff8aa0da1b40 Bug-Url: https://bugzilla.redhat.com/1117489 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com&gt; --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseGetAttachableDisksQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllAttachableDisksForVmQuery.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllAttachableDisksQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetFilteredAttachableDisksQuery.java R backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetAllAttachableDisksForVmQueryParameters.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetFilteredAttachableDisksParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 8 files changed, 140 insertions(+), 65 deletions(-) Approvals: Tomas Jelinek: Verified Tal Nisan: Looks good to me, approved Objections: oVirt Jenkins CI Server: Fails -- To view, visit https://gerrit.ovirt.org/36060 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If363cd92aa0071c88f2333294002ff8aa0da1b40 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com&gt; Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com&gt; Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com&gt; Gerrit-Reviewer: Candace Sheremeta <cshereme(a)redhat.com&gt; Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com&gt; Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com&gt; Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com&gt; Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com&gt; Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com&gt; Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server