Eliraz Levi has uploaded a new change for review.

Change subject: engine: short summ:slave's mac in Broker, to lower case
......................................................................

engine: short summ:slave's mac in Broker, to lower case

slave's mac in older vdsm versions, is in upper case, resulting
slave's mac is saved and repressent in upper case

Change-Id: I6ec2c04bca879c97835d95330c130f2a4ca16587
Bug-Url: https://bugzilla.redhat.com/1132422
Signed-off-by: Eliraz Levi <el...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/72/31872/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
index 4a16c47..fac2234 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
@@ -1436,7 +1436,9 @@
             iface.setMacAddress((String) nic.get("hwaddr"));
             // if we get "permhwaddr", we are a part of a bond and we use that 
as the mac address
             if (nic.get("permhwaddr") != null) {
-                iface.setMacAddress((String) nic.get("permhwaddr"));
+                // in older VDSM version, slave's Mac is in upper case
+                String macUpperCase = (String) nic.get("permhwaddr");
+                iface.setMacAddress(macUpperCase.toLowerCase());
             }
             if (StringUtils.isNotBlank((String) nic.get(VdsProperties.MTU))) {
                 iface.setMtu(Integer.parseInt((String) 
nic.get(VdsProperties.MTU)));


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

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

Reply via email to