Alona Kaplan has uploaded a new change for review.

Change subject: webadmin: Gateway is not defined after bonding the RHEVM 
interface.
......................................................................

webadmin: Gateway is not defined after bonding the RHEVM interface.

When detaching the network its parameters (boot protocol, address, subnet
and gateway- for managenemt network) are saved for the next attach.

The bug was that in case of managemnet network the gateway parameter was
always determined from the nicEntity.
But if the management network if over vlan it should be taken from the
vlanNic.

Change-Id: I7deff4305d04a6cfc943bb38cb795373ea7ff6e4
Bug-Url: https://bugzilla.redhat.com/923443
Signed-off-by: Alona Kaplan <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/90/13790/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java
index 74399a6..ab03b03 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java
@@ -134,7 +134,11 @@
         }
 
         if (isManagement()) {
-            netParams.setGateway(nicEntity.getGateway());
+            if (!hasVlan()) {
+                netParams.setGateway(nicEntity.getGateway());
+            } else {
+                netParams.setGateway(vlanNic.getEntity().getGateway());
+            }
         }
 
         getSetupModel().getNetworkToLastDetachParams().put(getName(), 
netParams);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7deff4305d04a6cfc943bb38cb795373ea7ff6e4
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