Alona Kaplan has uploaded a new change for review.

Change subject: webadmin: It is not possible add non-operational network to 
vNic of vm.
......................................................................

webadmin: It is not possible add non-operational network to vNic of vm.

Removed the if that avoided adding the non-operational networks to the
network list.

Change-Id: I6183a5abcddc343323b82c39111de9f2f1ba5166
Bug-Url: https://bugzilla.redhat.com/948160
Signed-off-by: Alona Kaplan <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/66/13666/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java
index 097d53f..d1c073a 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java
@@ -9,7 +9,6 @@
 import org.ovirt.engine.core.common.action.VdcReturnValueBase;
 import org.ovirt.engine.core.common.businessentities.VmBase;
 import org.ovirt.engine.core.common.businessentities.network.Network;
-import org.ovirt.engine.core.common.businessentities.network.NetworkStatus;
 import org.ovirt.engine.core.common.businessentities.network.VmInterfaceType;
 import 
org.ovirt.engine.core.common.businessentities.network.VmNetworkInterface;
 import org.ovirt.engine.core.common.queries.ConfigurationValues;
@@ -446,7 +445,7 @@
                 ArrayList<Network> networks = new ArrayList<Network>();
                 for (Network a : (ArrayList<Network>) result1)
                 {
-                    if (a.getCluster().getStatus() == 
NetworkStatus.OPERATIONAL && a.isVmNetwork())
+                    if (a.isVmNetwork())
                     {
                         networks.add(a);
                     }


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

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

Reply via email to