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 <sesub...@redhat.com> Gerrit-Reviewer: Ayal Baron <aba...@redhat.com> Gerrit-Reviewer: Livnat Peer <lp...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
