Change in ovirt-engine[master]: core: Auto-Recovery should check whether getVdsStats return...

mkublin at redhat.com mkublin at redhat.com
Mon Sep 24 10:04:58 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list