shwstppr commented on code in PR #11594:
URL: https://github.com/apache/cloudstack/pull/11594#discussion_r2367502322


##########
engine/schema/src/main/resources/META-INF/db/schema-42100to42200.sql:
##########
@@ -21,3 +21,33 @@
 
 -- Increase length of scripts_version column to 128 due to md5sum to sha512sum 
change
 CALL `cloud`.`IDEMPOTENT_CHANGE_COLUMN`('cloud.domain_router', 
'scripts_version', 'scripts_version', 'VARCHAR(128)');
+
+-- VMware to KVM migration improvements
+CREATE TABLE IF NOT EXISTS `cloud`.`import_vm_task`(
+    `id` bigint unsigned NOT NULL auto_increment COMMENT 'id',
+    `uuid` varchar(40),
+    `zone_id` bigint unsigned NOT NULL COMMENT 'Zone ID',
+    `account_id` bigint unsigned NOT NULL COMMENT 'Account ID',
+    `user_id` bigint unsigned NOT NULL COMMENT 'User ID',
+    `vm_id` bigint unsigned COMMENT 'VM ID',
+    `display_name` varchar(255) COMMENT 'Display VM Name',
+    `vcenter` varchar(255) COMMENT 'VCenter',
+    `datacenter` varchar(255) COMMENT 'VCenter Datacenter name',
+    `source_vm_name` varchar(255) COMMENT 'Source VM name on vCenter',
+    `convert_host_id` bigint unsigned COMMENT 'Convert Host ID',
+    `import_host_id` bigint unsigned COMMENT 'Import Host ID',
+    `step` varchar(20) NOT NULL COMMENT 'Importing VM Task Step',
+    `description` varchar(255) COMMENT 'Importing VM Task Description',
+    `duration` bigint unsigned COMMENT 'Duration in milliseconds for the 
completed tasks',
+    `created` datetime NOT NULL COMMENT 'date created',
+    `updated` datetime COMMENT 'date updated if not null',
+    `removed` datetime COMMENT 'date removed if not null',
+    PRIMARY KEY (`id`),
+    CONSTRAINT `fk_import_vm_task__zone_id` FOREIGN KEY 
`fk_import_vm_task__zone_id` (`zone_id`) REFERENCES `data_center`(`id`) ON 
DELETE CASCADE,
+    CONSTRAINT `fk_import_vm_task__account_id` FOREIGN KEY 
`fk_import_vm_task__account_id` (`account_id`) REFERENCES `account`(`id`) ON 
DELETE CASCADE,
+    CONSTRAINT `fk_import_vm_task__user_id` FOREIGN KEY 
`fk_import_vm_task__user_id` (`user_id`) REFERENCES `user`(`id`) ON DELETE 
CASCADE,
+    CONSTRAINT `fk_import_vm_task__vm_id` FOREIGN KEY 
`fk_import_vm_task__vm_id` (`user_id`) REFERENCES `vm_instance`(`id`) ON DELETE 
CASCADE,

Review Comment:
   Will need fixing



##########
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java:
##########
@@ -1422,6 +1432,22 @@ public boolean configure(final String name, final 
Map<String, Object> params) th
         return true;
     }
 
+    private void setConvertInstanceEnv(String convertEnvTmpDir, String 
convertEnvVirtv2vTmpDir) {
+        if (StringUtils.isAllBlank(convertEnvTmpDir, convertEnvVirtv2vTmpDir)) 
{
+            return;
+        }
+        if (StringUtils.isNotBlank(convertEnvTmpDir) && 
StringUtils.isNotBlank(convertEnvVirtv2vTmpDir)) {
+            convertInstanceEnv = new String[2];
+            convertInstanceEnv[0] = String.format("%s=%s", "TMPDIR", 
convertEnvTmpDir);
+            convertInstanceEnv[1] = String.format("%s=%s", "VIRT_V2V_TMPDIR", 
convertEnvTmpDir);

Review Comment:
   Will need fixing



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to