Maor Lipchuk has uploaded a new change for review.

Change subject: core: Add snapshot permission for live migration
......................................................................

core: Add snapshot permission for live migration

Add permission check for VM snapshot on live storage migration.

Change-Id: I6c0de2b0a7736ece173530f3c6f204bff503ab52
Signed-off-by: Maor Lipchuk <mlipc...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/76/37076/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
index ac4c4eb..feedf50 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java
@@ -166,6 +166,9 @@
             permissionList.add(new 
PermissionSubject(parameters.getTargetStorageDomainId(),
                     VdcObjectType.Storage,
                     ActionGroup.CREATE_DISK));
+            permissionList.add(new PermissionSubject(parameters.getVmId(),
+                    VdcObjectType.VM,
+                    ActionGroup.MANIPULATE_VM_SNAPSHOTS));
         }
 
         return permissionList;


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6c0de2b0a7736ece173530f3c6f204bff503ab52
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to