Eliraz Levi has uploaded a new change for review. Change subject: engine:slave's mac in Broker, to lower case ......................................................................
engine: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 <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/77/32577/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..7b5626c 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,10 @@ 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")); + //TODO remove when the minimal supported vdsm version is >=3.6 + // 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/32577 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6ec2c04bca879c97835d95330c130f2a4ca16587 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Eliraz Levi <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
