Change in ovirt-engine[master]: core: set vm owner on import vm like on add vm

Code Review gerrit at ovirt.org
Tue Aug 22 04:16:40 UTC 2017


>From Arik Hadas <ahadas at redhat.com>:

Arik Hadas has submitted this change and it was merged. ( https://gerrit.ovirt.org/77050 )

Change subject: core: set vm owner on import vm like on add vm
......................................................................


core: set vm owner on import vm like on add vm

In theory, the permission settings on import VM should be similar to that
on add VM. This patch applies what commit I7e814ab0 did for add VM to
import VM, in order to prevent a scenario in which users import VMs and
then lack the required permissions to manage them.

Change-Id: I4a8c63e8a5cce4c313fdeee8e90a8857faca11d0
Bug-Url: https://bugzilla.redhat.com/1451501
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java
1 file changed, 33 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Arik Hadas: Verified; Looks good to me, approved
  Michal Skrivanek: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4a8c63e8a5cce4c313fdeee8e90a8857faca11d0
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Sharon Gratch <sgratch at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud 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