nvazquez 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-436604927 Thanks @rafaelweingartner, the aim of this PR was just introducing the new strategy class as a placeholder so we could both extend its functionality on separate PRs. We can also close this PR and create the new file on both PRs and then rebase and resolve the conflicts when one of them gets merged as you mentioned, but thought this way would be easier to track changes and at the same time PRs won't depend on each other, they'll just add logic into existing new strategy.
---------------------------------------------------------------- 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