You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2021/01/19 11:38:26 UTC

[GitHub] [cloudstack] GutoVeronezi edited a comment on pull request #4570: Externalize KVM Agent's option to change migration thread timeout

GutoVeronezi edited a comment on pull request #4570:
URL: https://github.com/apache/cloudstack/pull/4570#issuecomment-762786813


   > @GutoVeronezi makes sense functionally and the code looks good except for the naming, why do you use `dest` in the names of the method and the setting? isn't this a local setting of the agent we are on at this moment?
   
   @DaanHoogland 
   
   Yes, it is local, but it refers to a migration from one host to the other, so it is verified if the VMs were correctly migrated from the source host to the `destination` host. The naming corresponds to the verification of the VM's DOM on the destination host.


----------------------------------------------------------------
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:
users@infra.apache.org