Kanagaraj M has posted comments on this change.

Change subject: webadmin: Show clusters with Virt service enabled only for VM
......................................................................


Patch Set 2:

If we can make the filtering more generic like fetching the 
DataCenters/Clusters based on their service types (virt/gluster/both), the same 
could be re-used in other places as well.

Like the user will be allowed to create a VM only in a virt supported cluster, 
gluster volume can be created only in gluster supported cluster. Currently, in 
the create volume dialog, we are manually filtering the cluster after fetching 
the full list from the server. This could be enhanced to use the generic filter 
mechanism.

--
To view, visit http://gerrit.ovirt.org/11395
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb63c7e2addf525bb527d94694a8dfc752d69154
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Libor Spevak <lspe...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Libor Spevak <lspe...@redhat.com>
Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to