Github user serg38 commented on the issue:

    https://github.com/apache/cloudstack/pull/1542
  
    @jburwell Creating Marvin test for this PR will be a huge undertaking. It 
would require building and configuring a hypervisor in a deployed VM and then 
creating VM in VM. Considering storage, environmental and network implication 
the test will be complicated and error prone. An alternative approach would 
require integrating vSphere SDK with Marvin python libraries and query vSphere 
for the actual nested virtualization settings from a deployed VM. The effort 
doesn't seem to be warranted for such a minor change. As long as standard 
integration suite is passing this shouldn't affect any existing functionality. 
What do you think?


---
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.
---

Reply via email to