Ayal Baron has posted comments on this change. Change subject: Introduce virt_service and gluster_service at cluster level ......................................................................
Patch Set 1: I would prefer that you didn't submit this We've already identified that going forward we will want to support other service types as well, esp. once we get rid of the pool and are able to manage host groups per VM. Adding a column to the db per service hardly seems scalable (changing db scheme for every service we add?) I think it would be better to add a mapping table (cluster / service). -- To view, visit http://gerrit.ovirt.org/2506 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75e63378c68458ec4230c3cb904e0ed2ac28a483 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Selvasundaram <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Livnat Peer <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Shireesh Anjal <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
