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 <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Libor Spevak <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to