Yair Zaslavsky has posted comments on this change. Change subject: engine: Fixing a possible bug at SPM election ......................................................................
Patch Set 1: Correct me if I'm wrong, but the code path of null at GetPrioritizedVdsInPool when called from gethostFromVds always occurs, this means that before patch all vds that were retrieved from DAO are returned by GetPrioritizedVdsInPool, including the vds that was represented bu currentVdsId - so now after the patch you're not returning it. Is this what you intended to do? -- To view, visit http://gerrit.ovirt.org/9309 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I44dfbb3a59c22e1b4ce39c996e159e1fd5047475 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@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
