Change in ovirt-engine[ovirt-engine-4.1]: core: Set vmId to the target VM ID in ImportVmCommandBase

Code Review gerrit at ovirt.org
Wed Aug 2 15:59:34 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged. ( https://gerrit.ovirt.org/80120 )

Change subject: core: Set vmId to the target VM ID in ImportVmCommandBase
......................................................................


core: Set vmId to the target VM ID in ImportVmCommandBase

Set vmId field in the ImportVmCommandBase command to the ID of the
target VM. This makes getVmIdRef() (that is used for audit logging) to
return the correct VM ID.

Change-Id: Ia545533d0ba471a01f2eef581f3fc244ea0276d9
Bug-Url: https://bugzilla.redhat.com/1477375
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Tomas Jelinek: Verified
  Jenkins CI: Passed CI tests
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia545533d0ba471a01f2eef581f3fc244ea0276d9
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list