Moti Asayag has uploaded a new change for review. Change subject: api: slave interfaces are being overridden ......................................................................
api: slave interfaces are being overridden When the client provides a bond with its slaves, the slaves are being reloaded from the engine, causing the information provided by the client to be overridden. The data that should not be set for slaves contain the network name and the boot protocol. The patch suggests an easy fix for the issue. A complete solution will be providing Bond entity when intended to define a bond, rather than a flat list of interfaces. Change-Id: I4f38bc399059ad19e93fcad0f360957636ac7045 Bug-Url: https://bugzilla.redhat.com/907240 Signed-off-by: Moti Asayag <[email protected]> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/71/11671/1 diff --git a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java index 2ec2c9c..654685d 100644 --- a/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java +++ b/backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java @@ -379,6 +379,11 @@ VdsNetworkInterface slaveIface = map(slave, slave.getId() == null ? lookupInterfaceByName(slave.getName()) : lookupInterfaceById(slave.getId())); slaveIface.setBondName(nic.getName()); + slaveIface.setNetworkName(null); + slaveIface.setBootProtocol(null); + slaveIface.setAddress(null); + slaveIface.setSubnet(null); + slaveIface.setGateway(null); ifaces.add(slaveIface); } } -- To view, visit http://gerrit.ovirt.org/11671 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4f38bc399059ad19e93fcad0f360957636ac7045 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
