CLOUDSTACK-9610: Disabled Host Keeps Being up status after unmanging cluster.

(cherry picked from commit 4b165f1)
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

Author: Priyank Parihar

engine/components-api/src/com/cloud/resource/ResourceManager.java

@@ -97,6 +97,8 @@ public interface ResourceManager extends ResourceService {
 
     public List<HostVO> listAllUpAndEnabledHosts(Host.Type type, Long clusterId, Long podId, long dcId);
 
+    public List<HostVO> listAllHosts(final Host.Type type, final Long clusterId, final Long podId, final long dcId);
+
     public List<HostVO> listAllHostsInCluster(long clusterId);
 
     public List<HostVO> listHostsInClusterByStatus(long clusterId, Status status);

server/src/com/cloud/resource/ResourceManagerImpl.java

@@ -1086,7 +1086,7 @@ public Cluster updateCluster(final Cluster clusterToUpdate, final String cluster
                 try {
                     cluster.setManagedState(Managed.ManagedState.PrepareUnmanaged);
                     _clusterDao.update(cluster.getId(), cluster);
-                    List<HostVO> hosts = listAllUpAndEnabledHosts(Host.Type.Routing, cluster.getId(), cluster.getPodId(), cluster.getDataCenterId());
+                    List<HostVO> hosts = listAllHosts(Host.Type.Routing, cluster.getId(), cluster.getPodId(), cluster.getDataCenterId());
                     for (final HostVO host : hosts) {
                         if (host.getType().equals(Host.Type.Routing) && !host.getStatus().equals(Status.Down) && !host.getStatus().equals(Status.Disconnected) &&
                                 !host.getStatus().equals(Status.Up) && !host.getStatus().equals(Status.Alert)) {
@@ -2494,6 +2494,22 @@ public List<HostVO> listAllUpAndEnabledHosts(final Type type, final Long cluster
         return sc.list();
     }
 
+    @Override
+    public List<HostVO> listAllHosts(final Type type, final Long clusterId, final Long podId, final long dcId) {
+        final QueryBuilder<HostVO> sc = QueryBuilder.create(HostVO.class);
+        if (type != null) {
+            sc.and(sc.entity().getType(), Op.EQ, type);
+        }
+        if (clusterId != null) {
+            sc.and(sc.entity().getClusterId(), Op.EQ, clusterId);
+        }
+        if (podId != null) {
+            sc.and(sc.entity().getPodId(), Op.EQ, podId);
+        }
+        sc.and(sc.entity().getDataCenterId(), Op.EQ, dcId);
+        return sc.list();
+    }
+
     @Override
     public List<HostVO> listAllUpAndEnabledNonHAHosts(final Type type, final Long clusterId, final Long podId, final long dcId) {
         final String haTag = _haMgr.getHaTag();

server/test/com/cloud/resource/MockResourceManagerImpl.java

@@ -359,6 +359,11 @@ public List<HostVO> listAllUpAndEnabledHosts(final Type type, final Long cluster
         return null;
     }
 
+    @Override
+    public List<HostVO> listAllHosts(final Type type, final Long clusterId, final Long podId, final long dcId) {
+        return null;
+    }
+
     /* (non-Javadoc)
      * @see com.cloud.resource.ResourceManager#listAllHostsInCluster(long)
      */