Change in ovirt-engine[master]: core: Auto-Recovery should check whether getVdsStats return...
Michael Kublin has submitted this change and it was merged.
Change subject: core: Auto-Recovery should check whether getVdsStats returns
'lastCheck<60' before it proclaims host as up (#844438)
......................................................................
core: Auto-Recovery should check whether getVdsStats returns 'lastCheck<60'
before it proclaims host as up (#844438)
https://bugzilla.redhat.com/844438
The described bug is a private case of regular activate host operation.
The problem is that the condition for moving host to NonOperational is different for those
that move host to Active state.
Solution is during IniVdsOnUpCommand perform getVdsStats in order to get info about
storage domains that host can see,
if all Active domins in cluster is seen by host it will be moved Active, otherwise it will
be left as NonOperational
with reason: STORAGE_DOMAIN_UNREACHABLE
Change-Id: Id88ef24829d5fae61dddd34f0265ae3132672783
Signed-off-by: Michael Kublin <mkublin(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
M
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
6 files changed, 83 insertions(+), 8 deletions(-)
Approvals:
Michael Kublin: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/8131
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id88ef24829d5fae61dddd34f0265ae3132672783
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>