cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [1/3] git commit: updated refs/heads/master to 3c2e8b9
Date Mon, 22 Aug 2016 11:06:33 GMT
Repository: cloudstack
Updated Branches:
  refs/heads/master b6bf093a3 -> 3c2e8b929

CLOUDSTACK-9463: Fix dynamic-roles migrate script for old format

The old format included the full class name such as:;1

The migration script did not consider this format and fails. With this fix
the migration script will process both the formats, including processing a file with mixed format, for example:

    $ cat
    ### Account commands

    $ python scripts/util/ -d -f
    Apache CloudStack Role Permission Migration Tool
    (c) Apache CloudStack Authors and the ASF, under the Apache License, Version 2.0

    Running this migration tool will remove any default-role permissions from cloud.role_permissions.
Do you want to continue? [y/N]y
    The file has been deprecated and moved at:
    Running SQL query: DELETE FROM `cloud`.`role_permissions` WHERE `role_id` in (1,2,3,4);
    Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`,
`permission`, `sort_order`) values (UUID(), 1, '*', 'ALLOW', 0);
    Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`,
`permission`, `sort_order`) values (UUID(), 2, 'deleteAccount', 'ALLOW', 0);
    Running SQL query: INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`,
`permission`, `sort_order`) values (UUID(), 2, 'markDefaultZoneForAccount', 'ALLOW', 1);
    Static role permissions from have been migrated into the db
    Running SQL query: UPDATE `cloud`.`configuration` SET value='true' where name='dynamic.apichecker.enabled'
    Dynamic role based API checker has been enabled!

Signed-off-by: Rohit Yadav <>


Branch: refs/heads/master
Commit: f699fd4a15c8e58c534e98d8436beedf3b54fdc5
Parents: bdc409c
Author: Rohit Yadav <>
Authored: Fri Aug 19 17:27:20 2016 +0530
Committer: Rohit Yadav <>
Committed: Fri Aug 19 17:33:07 2016 +0530

 scripts/util/ | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/util/ b/scripts/util/
index d2356fb..cbb83f9 100755
--- a/scripts/util/
+++ b/scripts/util/
@@ -107,10 +107,10 @@ def main():
     apiMap = {}
     with open(options.commandsfile) as f:
         for line in f.readlines():
-            if not line or line == '' or line == '\n' or line.startswith('#'):
+            if not line or line == '' or line == '\n' or line == '\r\n' or line.startswith('#'):
             name, value = line.split('=')
-            apiMap[name.strip()] = value.strip()
+            apiMap[name.strip()] = value.strip().split(';')[-1]
     # Rename and deprecate old file
     if not dryrun:

View raw message