rafaelweingartner commented on a change in pull request #2578: api: add command to list management servers URL: https://github.com/apache/cloudstack/pull/2578#discussion_r225502221
########## File path: engine/schema/src/main/resources/META-INF/db/schema-41120to41200.sql ########## @@ -34,4 +34,7 @@ INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`, `permission`, INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 3, 'moveNetworkAclItem', 'ALLOW', 302) ON DUPLICATE KEY UPDATE rule=rule; INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`, `permission`, `sort_order`) values (UUID(), 4, 'moveNetworkAclItem', 'ALLOW', 260) ON DUPLICATE KEY UPDATE rule=rule; -UPDATE `cloud`.`async_job` SET `removed` = now() WHERE `removed` IS NULL; \ No newline at end of file +UPDATE `cloud`.`async_job` SET `removed` = now() WHERE `removed` IS NULL; + +-- New column for listManagementServers API call +ALTER TABLE `mshost` ADD COLUMN `uuid` varchar(40) AFTER `name`; Review comment: Ok, but here I still have a question. I did not see any code to populate this `uuid` column in `mshost` table when you start to use this feature. I mean, after we upgrade the ACS to 4.12, how will we populate this `uuid` field for MShosts that already exist? Also, ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
