Sergey Gotliv has posted comments on this change. Change subject: engine: fix null pointer in VdsUpdateRunTimeInfo ......................................................................
Patch Set 2: Looks good to me, but someone else must approve For the future, I would consider to perform sync between VMs running on host and VMs from the DB first in the separate method, so all other methods in this class won't need to check the nullness. But for now this is good enough. -- To view, visit http://gerrit.ovirt.org/17297 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I07ba7082196441ad6bd5b66436e3c944c4abec21 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslom...@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com> Gerrit-Reviewer: Noam Slomianko <nslom...@redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
