Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/954#issuecomment-150897664
I was concerned the code change is against the initial design and we may be
doing it without understanding all the implications. But, since we dont have
any better solution and the current set of tests we have are positive with the
change, I would :+1: this. Though, we should keep in mind that this may lead
to some regressions like CLOUDSTACK-3681 or CLOUDSTACK-6478
@mike-tutkowski Can you also test if the vmware snapshot-from-template
issue you faced is fixed by this?
@ustcweizhou Thanks for fixing it very quickly. A little description would
have saved me sometime in trying to find the root cause :)
---
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.
---
