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 2018/11/06 21:23:39 UTC

[GitHub] rafaelweingartner commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations

rafaelweingartner commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations
URL: https://github.com/apache/cloudstack/pull/3004#issuecomment-436413549
 
 
   @nvazquez, I am with @DaanHoogland here. Can you explain further the need for this PR? I mean, this class that you introduce here seems to be something that is already introduced in one of those PRs that you mention. Therefore, if someone else needs it, or if the PRs are somehow dependent, I would say, let’s work together to get the complete PR in, and not just some part of it. Then, after getting the first one, we can work to get the second one.
   

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


With regards,
Apache Git Services