rhtyd commented on a change in pull request #3271: [WIP DO NOT MERGE] [Vmware] Allow configuring appliances on the VM instance wizard when OVF properties are available URL: https://github.com/apache/cloudstack/pull/3271#discussion_r274288645
########## File path: engine/schema/src/main/resources/META-INF/db/schema-41200to41300.sql ########## @@ -19,3 +19,29 @@ -- Schema upgrade from 4.12.0.0 to 4.13.0.0 --; +CREATE TABLE `cloud`.`template_ovf_properties` ( + `id` bigint(20) unsigned NOT NULL AUTO_INCREMENT, + `template_id` bigint(20) unsigned NOT NULL, + `key` VARCHAR(100) NOT NULL, + `type` VARCHAR(45) DEFAULT NULL, + `value` VARCHAR(100) DEFAULT NULL, + `qualifiers` TEXT DEFAULT NULL, + `user_configurable` TINYINT(1) NOT NULL DEFAULT '0', + `label` TEXT DEFAULT NULL, + `description` TEXT DEFAULT NULL, + `removed` datetime DEFAULT NULL, + PRIMARY KEY (`id`), + CONSTRAINT `fk_template_ovf_properties__template_id` FOREIGN KEY (`template_id`) REFERENCES `vm_template`(`id`) +) ENGINE=InnoDB DEFAULT CHARSET=utf8; + +CREATE TABLE `cloud`.`user_vm_ovf_properties` ( Review comment: @nvazquez can't we not reuse the _details table for this? Essentially the properties can be treated as properties of the template and the vm instance? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
