nvazquez commented on a change in pull request #2983: KVM live storage 
migration intra cluster from NFS source and destination
URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229305185
 
 

 ##########
 File path: engine/schema/src/main/resources/META-INF/db/schema-41120to41200.sql
 ##########
 @@ -37,4 +37,4 @@ INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, 
`rule`, `permission`,
 UPDATE `cloud`.`async_job` SET `removed` = now() WHERE `removed` IS NULL;
 
 -- PR#1448 update description of 
'execute.in.sequence.network.element.commands' parameter to reflect an unused 
command that has been removed. The removed class command is 'UserDataCommand'.
-update `cloud`.`configuration` set description = 'If set to true, 
DhcpEntryCommand, SavePasswordCommand, VmDataCommand will be synchronized on 
the agent side. If set to false, these commands become asynchronous. Default 
value is false.' where name = 'execute.in.sequence.network.element.commands'; 
\ No newline at end of file
+update `cloud`.`configuration` set description = 'If set to true, 
DhcpEntryCommand, SavePasswordCommand, VmDataCommand will be synchronized on 
the agent side. If set to false, these commands become asynchronous. Default 
value is false.' where name = 'execute.in.sequence.network.element.commands';
 
 Review comment:
   Had previously included an update sql on the `hypervisor_cappabilities` to 
enable live storage motion on KVM. But due to conflicting versions on qemu just 
decided to remove it and must have removed an end of line or extra space, will 
sort it out

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to