Change in ovirt-engine[master]: core: dont save vdsmVm to db

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Jun 25 11:39:32 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list