Sahina Bose has uploaded a new change for review.

Change subject: engine: Report domain monitoring for virt nodes only
......................................................................

engine: Report domain monitoring for virt nodes only

Storage domain monitoring is required only for
virtualization nodes (not for gluster nodes)

Change-Id: I8731a81de756cbe0bb221f205c2a6d50c015b86b
Bug-Url: https://bugzilla.redhat.com/1105513
Signed-off-by: Sahina Bose <[email protected]>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/48/31448/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
index 984871e..286ca3b 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java
@@ -46,14 +46,15 @@
     static final VDSStatus reportingVdsStatus = VDSStatus.Up;
 
     /**
-     * process received domain monitoring information from a given vds if 
necessary (according to it's status).
+     * process received domain monitoring information from a given vds if 
necessary (according to it's status
+     * and if it's a virtualization node).
      * @param vds
      * @param storagePoolId
      * @param vdsDomainData
      */
     public static void updateVdsDomainsData(VDS vds, Guid storagePoolId,
             ArrayList<VDSDomainsData> vdsDomainData) {
-        if (vds.getStatus() == reportingVdsStatus) {
+        if (vds.getStatus() == reportingVdsStatus && 
vds.getVdsGroupSupportsVirtService()) {
             IrsProxyData proxy = _irsProxyData.get(storagePoolId);
             if (proxy != null) {
                 proxy.updateVdsDomainsData(vds.getId(), vds.getName(), 
vdsDomainData);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8731a81de756cbe0bb221f205c2a6d50c015b86b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to