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/18 15:04:23 UTC

[GitHub] [cloudstack] DaanHoogland commented on pull request #4570: Externalize KVM Agent's option to change migration thread timeout

DaanHoogland commented on pull request #4570:
URL: https://github.com/apache/cloudstack/pull/4570#issuecomment-762305692


   @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?


----------------------------------------------------------------
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