Omer Frenkel has submitted this change and it was merged.
Change subject: core: dont check vmManager timestamp if it wasnt set
......................................................................
core: dont check vmManager timestamp if it wasnt set
when engine starts, the first monitoring cycle will always skip the
current running vms because the VmManager objects for them are created
only after fetching the information from vdsm, so the time check will
always fail.
the check is changed to verify that some value has been set,
and if not it considered ok to do the monitoring cycle.
Change-Id: I6ced6416b4015b86aeb75371224ad68dc1f5b74c
Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com>
---
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmManager.java
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java
2 files changed, 3 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/42497
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6ced6416b4015b86aeb75371224ad68dc1f5b74c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org