Sahina Bose has uploaded a new change for review.

Change subject: engine: Host activate -check gluster availability
......................................................................

engine: Host activate -check gluster availability

If cluster supports gluster service, need to
check if gluster is running, while activating host.
If gluster command fails on server, the host state
should not be changed to UP on activate.

Change-Id: If673e32ed6480187ddcf6f9ecf8f0007e203b98d
Bug-Url: https://bugzilla.redhat.com/975382
Signed-off-by: Sahina Bose <sab...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
1 file changed, 12 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/98/16898/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
index 71aad99..5abee29 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
@@ -29,6 +29,7 @@
 import org.ovirt.engine.core.common.vdscommands.VDSCommandType;
 import org.ovirt.engine.core.common.vdscommands.VDSReturnValue;
 import 
org.ovirt.engine.core.common.vdscommands.VdsIdAndVdsVDSCommandParametersBase;
+import org.ovirt.engine.core.common.vdscommands.VdsIdVDSCommandParametersBase;
 import org.ovirt.engine.core.compat.Guid;
 import org.ovirt.engine.core.compat.Version;
 import org.ovirt.engine.core.dal.dbbroker.DbFacade;
@@ -573,6 +574,17 @@
                 }
             }
 
+            if (vds.getVdsGroupSupportsGlusterService()) {
+                // check if gluster is running
+                VDSReturnValue returnValue = 
ResourceManager.getInstance().runVdsCommand(VDSCommandType.GlusterServersList,
+                        new VdsIdVDSCommandParametersBase(vds.getId()));
+                if (!returnValue.getSucceeded()) {
+                    vds.setStatus(VDSStatus.NonOperational);
+                    
vds.setNonOperationalReason(NonOperationalReason.GLUSTER_COMMAND_FAILED);
+                    returnStatus = vds.getStatus();
+                }
+            }
+
             // We process the software capabilities.
             VDSStatus oldStatus = vds.getStatus();
             monitoringStrategy.processSoftwareCapabilities(vds);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If673e32ed6480187ddcf6f9ecf8f0007e203b98d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sab...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to