Change in ovirt-engine[master]: core: REST-API request of VM's NIC with non-exists network

emesika at redhat.com emesika at redhat.com
Thu Jun 7 15:25:25 UTC 2012


Eli Mesika has submitted this change and it was merged.

Change subject: core: REST-API request of VM's NIC with non-exists network
......................................................................


core: REST-API request of VM's NIC with non-exists network

REST-API request of VM's NIC with non-exists network returns error
instead of the NIC's structure XML

This patch handles interface invalid network or non vm network and if
validation failed the network name will be an empty string and the user
will be asked to set the correct network upon running the VM

Change-Id: I50abd4319dbeaf6fa272bf9529bf4b1e651c8dcd
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmTemplateCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/VmInterfaceManager.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
4 files changed, 34 insertions(+), 11 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I50abd4319dbeaf6fa272bf9529bf4b1e651c8dcd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>



More information about the Engine-commits mailing list