Change in ovirt-engine[master]: restapi: Don't set bootproto on VLAN attach to NIC

masayag at redhat.com masayag at redhat.com
Sun Aug 18 10:04:58 UTC 2013


Moti Asayag has submitted this change and it was merged.

Change subject: restapi: Don't set bootproto on VLAN attach to NIC
......................................................................


restapi: Don't set bootproto on VLAN attach to NIC

Setting the boot protocol on attaching a network to a host's NIC causes
the new VLAN to get the same boot parameters as the NIC itself, which
makes little sense as the VLAN is a different L2 network and it's boot
protocol info should be set accordingly and not automatically.

Change-Id: I78b415592e2bc1a9fea901cadb3ee928f2935a14
Bug-Url: https://bugzilla.redhat.com/980486
Signed-off-by: Mike Kolesnik <mkolesni at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResourceTest.java
3 files changed, 41 insertions(+), 8 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I78b415592e2bc1a9fea901cadb3ee928f2935a14
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list