Change in ovirt-engine[master]: engine: old SetupNetworks, NPE when attaching network to ho...

masayag at redhat.com masayag at redhat.com
Tue Sep 8 13:50:30 UTC 2015


Moti Asayag has submitted this change and it was merged.

Change subject: engine:  old SetupNetworks, NPE when attaching network to host NIC
......................................................................


engine:  old SetupNetworks, NPE when attaching network to host NIC

NPE in the follwing cases-
1. Just the name of the attached network is passed (not the id).
2. Just the name of the nic is passed.
3. The 'vdsId' is not set on the passed host nic.

Change-Id: I1242f13a34df4bbdd7eee1e094cc02f28105585d
Bug-Url: https://bugzilla.redhat.com/1260430
Signed-off-by: Alona Kaplan <alkaplan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelper.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelperTest.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SetupNetworksVDSCommand.java
3 files changed, 51 insertions(+), 17 deletions(-)

Approvals:
  Alona Kaplan: Verified
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/45814
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1242f13a34df4bbdd7eee1e094cc02f28105585d
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list