Change in ovirt-engine[master]: engine: Hot-unplug fails when unplug nic and changing MAC ad...

alkaplan at redhat.com alkaplan at redhat.com
Wed Feb 27 13:19:19 UTC 2013


Alona Kaplan has submitted this change and it was merged.

Change subject: engine: Hot-unplug fails when unplug nic and changing MAC address.
......................................................................


engine: Hot-unplug fails when unplug nic and changing MAC address.

Hot-unplug fails when trying to unplug and change MAC address for nic
under running VM.
When unplugging a nic of a running vm-
1. The db is updated.
2. Unplug action is sent to the host.

It means that the vdsm gets unplug with the new mac address (although the
vnic is defined on the host with the old one).
To solve this issue the old mac address should be sent to the vdsm.

Change-Id: I185e5572501ea86443a93f4fdf309e0e05520646
Bug-Url: https://bugzilla.redhat.com/904840
Signed-off-by: Alona Kaplan <alkaplan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/AbstractVmInterfaceCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/ActivateDeactivateVmNicCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/UpdateVmInterfaceCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ActivateDeactivateVmNicParameters.java
4 files changed, 26 insertions(+), 7 deletions(-)

Approvals:
  Alona Kaplan: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I185e5572501ea86443a93f4fdf309e0e05520646
Gerrit-PatchSet: 2
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: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Muli Salem <msalem at redhat.com>



More information about the Engine-commits mailing list