Change in ovirt-engine[master]: HE: fixed the vm nic changing it's status to unplugged.

rgolan at redhat.com rgolan at redhat.com
Tue Mar 29 15:18:51 UTC 2016


Roy Golan has submitted this change and it was merged.

Change subject: HE: fixed the vm nic changing it's status to unplugged.
......................................................................


HE: fixed the vm nic changing it's status to unplugged.

The fix is to use the nic device id that is reported from the hosted engine
vm when importing it and not to generate a random id.

Added unitests.

Organized imports.

Change-Id: I0987418d33b61009cbd443e5b6c82872b20242d5
Bug-Url: https://bugzilla.redhat.com/1311151
Signed-off-by: Jenny Tokar <jtokar at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectBuilderTest.java
2 files changed, 42 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Roy Golan: Looks good to me, approved
  Jenny Tokar: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0987418d33b61009cbd443e5b6c82872b20242d5
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list