Github user anshul1886 commented on the pull request:
https://github.com/apache/cloudstack/pull/1176#issuecomment-161931232
@ustcweizhou I differ on this. We should not make HypervisorGuruManagerImpl
complex by adding these kind of conditions as nobody is expert of all
hypervisors and can make errors as happened in this case. This will keep the
core code simpler.
In XenServer after version 6.2 with hotfixes we start handling creating
template from snapshot in Hypervisor agent code. So the change is if version is
6.1 or 6.2 without hot fix then send the command to SSVM otherwise send to
hypervisor.
---
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.
---
