Change in ovirt-engine[ovirt-engine-3.4]: core: change domains status to unknown when there are no rep...

[oourfali at redhat.com]

Oved Ourfali has submitted this change and it was merged.

Change subject: core: change domains status to unknown when there are no reporting hosts
......................................................................


core: change domains status to unknown when there are no reporting hosts

Currently the monitored pool domains 'change' their statuses according to
the reports received by the pool hosts that are in status UP.

When there are no reporting hosts - the status of the pool domains that are
monitored should be changed to unknown, because otherwise nothing will trigger
the status change. In that case (an example) we might be left with a domain
in 'active' status although there are no reporting hosts.

Change-Id: I8091a4864711aeccee41effb1bc7d9823a1870c7
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=975742
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SetStoragePoolStatusCommand.java
D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/StoragePoolDomainHelper.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/storage/StoragePoolDomainHelper.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/ReconstructMasterVDSCommand.java
5 files changed, 90 insertions(+), 48 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, approved
  Liron Ar: Verified



-- 
To view, visit http://gerrit.ovirt.org/26077
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8091a4864711aeccee41effb1bc7d9823a1870c7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server