Tal Nisan has submitted this change and it was merged.

Change subject: core: hosts from gluster clusters shouldn't be considered
......................................................................


core: hosts from gluster clusters shouldn't be considered

When moving a domain to maintenance or when handling problematic
domain, hosts from gluster clusters were considered although we don't
use any monitoring from those hosts which prevents hosts from moving to
maintenance or from properly handle a problematic domain.

This patch fixes it by loading only the hosts from virt service (which
we collect domain monitoring information from) instead of loading all
the hosts.

Change-Id: I2b89785529ec00114065e350138e05d6c966df44
Bug-Url: https://bugzilla.redhat.com/1186687
Signed-off-by: Liron Aravot <[email protected]>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAO.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsDAOTest.java
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java
M packaging/dbscripts/vds_sp.sql
6 files changed, 30 insertions(+), 6 deletions(-)

Approvals:
  Eli Mesika: Looks good to me, but someone else must approve
  Allon Mureinik: Looks good to me, approved
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2b89785529ec00114065e350138e05d6c966df44
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Liron Aravot <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to