Federico Simoncelli has uploaded a new change for review.

Change subject: db: fix upgrade for app mode to roles
......................................................................

db: fix upgrade for app mode to roles

Role UserTemplateBasedVm was missing from the update list always
triggering the following error:

ERROR:  column "app_mode" contains null values

Change-Id: Ic5ebbba79deb6de091d27621196f361fd4fd3caf
Signed-off-by: Federico Simoncelli <fsimo...@redhat.com>
---
M packaging/dbscripts/upgrade/03_03_0810_add_app_mode_to_roles.sql
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/77/18777/1

diff --git a/packaging/dbscripts/upgrade/03_03_0810_add_app_mode_to_roles.sql 
b/packaging/dbscripts/upgrade/03_03_0810_add_app_mode_to_roles.sql
index 85d044f..03b1ecb 100644
--- a/packaging/dbscripts/upgrade/03_03_0810_add_app_mode_to_roles.sql
+++ b/packaging/dbscripts/upgrade/03_03_0810_add_app_mode_to_roles.sql
@@ -9,7 +9,7 @@
 select fn_db_add_column('roles', 'app_mode', 'INTEGER');
 
 UPDATE roles
-SET app_mode = (CASE WHEN name in ('UserRole', 'PowerUserRole', 
'DataCenterAdmin', 'StorageAdmin', 'UserVmManager', 'VmPoolAdmin', 
'TemplateAdmin', 'TemplateUser', 'QuotaConsumer', 'TemplateOwner', 
'DiskOperator', 'DiskCreator', 'VmCreator', 'TemplateCreator', 
'VnicProfileUser')
+SET app_mode = (CASE WHEN name in ('UserRole', 'PowerUserRole', 
'DataCenterAdmin', 'StorageAdmin', 'UserVmManager', 'VmPoolAdmin', 
'TemplateAdmin', 'TemplateUser', 'QuotaConsumer', 'TemplateOwner', 
'DiskOperator', 'DiskCreator', 'VmCreator', 'TemplateCreator', 
'VnicProfileUser', 'UserTemplateBasedVm')
                                THEN 1
                                WHEN name='GlusterAdmin'
                                THEN 2


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

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

Reply via email to