Omer Frenkel has submitted this change and it was merged. Change subject: core: dont save vdsmVm to db ...................................................................... core: dont save vdsmVm to db we should not save the vdsmVm to the db, as it doesnt contain the host-id that the vm runs on. also, it might initialize some information that exists in the db but not received from vdsm. changed prepareGuestAgentNetworkDevicesForUpdate method to use the right object (dbVm). Change-Id: Ie12c02f00eced21e4203235f29b94d62507bedc0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1230082 Signed-off-by: Omer Frenkel <ofrenkel@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmAnalyzer.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42552 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie12c02f00eced21e4203235f29b94d62507bedc0 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: automation@ovirt.org