Change in ovirt-engine[ovirt-engine-3.6]: core: fix events handling after host reboot

tnisan at redhat.com tnisan at redhat.com
Tue Sep 8 13:56:19 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: core: fix events handling after host reboot
......................................................................


core: fix events handling after host reboot

The engine filters events if it processed newer events already.
It compares the timestamp on each events with the timestamp of
the latest event that came from the same host and has been
processed.

VDSM uses monotonic time for the timestamp on the status
events. The engine relay on the timestamp being monotonic.
However, the timestamp is not monotonic after the host reboots.

The solution is to clear the last host-id that the latest event
that has been processed came from if that host has been
rebooted. That way, we won't use timestamp that has been
received before the host has been rebooted and the next event
will be processed.

Change-Id: Iba99b28312e1b729cd48dda80bac464f32ba7828
Bug-Url: https://bugzilla.redhat.com/1255033
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmManager.java
3 files changed, 16 insertions(+), 0 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iba99b28312e1b729cd48dda80bac464f32ba7828
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Arik Hadas <ahadas 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: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list