GutoVeronezi commented on code in PR #7094: URL: https://github.com/apache/cloudstack/pull/7094#discussion_r1072763279
########## engine/schema/src/main/resources/META-INF/db/schema-41720to41800.sql: ########## @@ -1035,3 +1035,16 @@ WHERE role_id = (SELECT id FROM `cloud`.`roles` WHERE name = 'Read-Only User - INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`, `permission`) SELECT UUID(), `roles`.`id`, 'isAccountAllowedToCreateOfferingsWithTags', 'ALLOW' FROM `cloud`.`roles` WHERE `role_type` = 'DomainAdmin'; + +--- Create table for handling allowed console session temporally #7094 + +CREATE TABLE IF NOT EXISTS `cloud`.`console_session` ( + `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT PRIMARY KEY, + `uuid` varchar(40) NOT NULL COMMENT 'UUID generated for the session', + `created` datetime NOT NULL COMMENT 'When the session was created', + `account_id` bigint(20) unsigned NOT NULL COMMENT 'Account who generated the session' references cloud.account(id), Review Comment: @weizhouapache, I found out the reason:  https://mariadb.com/kb/en/foreign-keys/ The syntax I firstly used would do nothing (neither warn the user :man_shrugging:) for environments with MariaDB version previous to 10.5. I changed it to work with the previous syntax (https://github.com/apache/cloudstack/pull/7094/commits/6173aa0896e42df81fcae5e4c806c9f56ad54b08), thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
