nvazquez 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_r274762544
########## 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: Fair enough, but we'll need to distinguish them from other VM details. Prepending a predefined string to each key? ---------------------------------------------------------------- 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
