Michael Kublin has posted comments on this change. Change subject: core:AddVdsSpmIdCommand throw exception when... ......................................................................
Patch Set 1: I would prefer that you didn't submit this The logic in bll is done in order to fill missing ids. For example, if we have spm ids 1,2,4 a next id should be 3. It is done because of the number of ids is limited and by simple increasing spm_id we are reducing a number of host in data centre, when some host were removed -- To view, visit http://gerrit.ovirt.org/11271 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7f8aa6b6cb3dfc1508a99c53553afc2e15009b68 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Liron Aravot <lara...@redhat.com> Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
