Alona Kaplan has uploaded a new change for review.

Change subject: webadmin: [DC Guide me] Host configured in one of the clusters 
should be displayed in the others
......................................................................

webadmin: [DC Guide me] Host configured in one of the clusters should
be displayed in the others

https://bugzilla.redhat.com/856504

- If the host is configured in one of the dc's clusters it should
be displayed in the others.
- If there is at least one host in the dc, the select host should be in
the optional area.
(There is no need to check if the host is supported by any cluster).

Change-Id: I800345a112aff8b4c913d9eed93e2f70d1f560d0
Signed-off-by: Alona Kaplan <alkap...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
1 file changed, 2 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/33/8333/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
index c0b59f8..7ea7766 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
@@ -275,14 +275,12 @@
         ArrayList<VDS> upHosts = new ArrayList<VDS>();
         for (VDS vds : allHosts)
         {
-            if (Linq.IsClusterItemExistInList(clusters, vds.getvds_group_id())
-                && doesHostSupportAnyCluster(clusters, vds))
+            if (Linq.IsClusterItemExistInList(clusters, vds.getvds_group_id()))
             {
                 hosts.add(vds);
             }
 
-            if ((!Linq.IsHostBelongsToAnyOfClusters(clusters, vds))
-                && (vds.getstatus() == VDSStatus.Maintenance || 
vds.getstatus() == VDSStatus.PendingApproval)
+            if ((vds.getstatus() == VDSStatus.Maintenance || vds.getstatus() 
== VDSStatus.PendingApproval)
                 && doesHostSupportAnyCluster(clusters, vds))
             {
                 availableHosts.add(vds);


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

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

Reply via email to