Oved Ourfali has uploaded a new change for review.

Change subject: core: Add UserVmManager to VM PowerUsers on upgrade (#883365)
......................................................................

core: Add UserVmManager to VM PowerUsers on upgrade (#883365)

https://bugzilla.redhat.com/883365

Recently the PowerUser was changed to be a creator role.
Thus, users with PowerUser on a VM can't run it, which doesn't make much
sense.
This upgrade script fixes that issue by adding a UserVmManager
permission on such VMs to the user which had direct PowerUser on them.

Change-Id: Id0d51bc27e9f6d5318744a73fbd1ed870cccf85d
Signed-off-by: Oved Ourfali <[email protected]>
---
A 
backend/manager/dbscripts/upgrade/03_02_0080_add_user_vm_manager_to_vm_power_users.sql
1 file changed, 18 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/83/10183/1

diff --git 
a/backend/manager/dbscripts/upgrade/03_02_0080_add_user_vm_manager_to_vm_power_users.sql
 
b/backend/manager/dbscripts/upgrade/03_02_0080_add_user_vm_manager_to_vm_power_users.sql
new file mode 100644
index 0000000..d490511
--- /dev/null
+++ 
b/backend/manager/dbscripts/upgrade/03_02_0080_add_user_vm_manager_to_vm_power_users.sql
@@ -0,0 +1,18 @@
+-- Inserting a UserVmManager permission on every PowerUser permission directly 
on a VM
+INSERT INTO permissions (id, role_id, ad_element_id, object_id, object_type_id)
+      (SELECT uuid_generate_v1(),
+             'def00006-0000-0000-0000-def000000006',
+             ad_element_id,
+             object_id,
+             object_type_id
+       FROM permissions perm1
+       WHERE role_id = '00000000-0000-0000-0001-000000000002'
+       AND object_type_id = 2 AND NOT EXISTS (
+           SELECT *
+           FROM permissions perm2
+           WHERE role_id = 'def00006-0000-0000-0000-def000000006'
+           AND object_type_id = 2
+           AND perm1.object_id = perm2.object_id
+           AND perm1.ad_element_id = perm2.ad_element_id
+       )
+);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id0d51bc27e9f6d5318744a73fbd1ed870cccf85d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to