Github user sateesh-chodapuneedi commented on the pull request:
https://github.com/apache/cloudstack/pull/815#issuecomment-212845624
>Purpose of doing this change is if some user intentionally want to do
migration from lower to higher then he can do it via API(from UI he can not do
this). Other way around hypervisor will not allow him.
@priyankparihar Even if user intentionally tries to perform migration via
API, ACS should fail the attempt after checking for version check condition,
rather than leaving this to hypervisor. Construct the condition per hypervisor
documentation and simply fail it with information/documentation references, if
required.
Also leaving it to hypervisor may be costlier, this being, storage
migration task, is an async job which increase async job count to be
tracked/managed by management server and also may involve multiple hypervisor
API calls, even for failure cases. It's good to reduce the jobs and API calls
if the outcome is known definitively in advance.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
