winterhazel commented on code in PR #8339:
URL: https://github.com/apache/cloudstack/pull/8339#discussion_r1425769329
##########
engine/schema/src/main/resources/META-INF/db/schema-41810to41900.sql:
##########
@@ -314,3 +314,8 @@ CREATE TABLE `cloud_usage`.`bucket_statistics` (
`size` bigint unsigned COMMENT 'total size of bucket objects',
PRIMARY KEY(`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;
+
+-- Add remover account ID to quarantined IPs table.
+ALTER TABLE `cloud`.`quarantined_ips`
+ ADD COLUMN `remover_account_id` bigint(20) unsigned DEFAULT NULL COMMENT
'ID of the account that removed the IP from quarantine, foreign key to
`account` table',
+ ADD CONSTRAINT `fk_quarantined_ips__remover_account_id` FOREIGN
KEY(`remover_account_id`) REFERENCES `cloud`.`account`(`id`);
Review Comment:
Yeah, it is better to have changes like this being idempotent. I will modify
the upgrade script.
--
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]
