Change in ovirt-engine[master]: core: events refresher - handle migrating and external vms

ofrenkel at redhat.com ofrenkel at redhat.com
Sun Jul 12 06:16:20 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: core: events refresher - handle migrating and external vms
......................................................................


core: events refresher - handle migrating and external vms

currently the events refresher ignores external vms (vm not in db)
and wrong handling vms that are in 'migration destination'.

the vms-monitoring expects the dbVm to be null on these cases,
and knows to handle migration destination and external vms.
this patch sends the right data to the monitoring.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1238114
Change-Id: If861153fb8893bcf35044d4371794ca58d05f366
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/EventVMStatsRefresher.java
1 file changed, 19 insertions(+), 14 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Omer Frenkel: Verified
  Arik Hadas: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/43219
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If861153fb8893bcf35044d4371794ca58d05f366
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list