Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1642
  
    @mike-tutkowski the background makes sense.  I would like to see that 
comment amended to commit message.  I have the following outstanding 
questions/items:
    
    1. Is there a JIRA ticket for this enhancement?  If not, please create one. 
 Also, please reference it in the commit message.
    1. Test results for the following test scenarios:
        * [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) + the new test integration test suite added for this PR against 
XenServer using Managed Storage for System VMs
        * [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) against XenServer using Unmanaged Storage for System VMs
        * [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) against VMware using Unmanaged Storage for System VMs
        * [ ] Smoke test suite + system VM integration test suites (SSVM, CPVM, 
and VR) against KVM using Unmanaged Storage for System VMs
    
    As I mentioned previously, these tests can be run across separate labs.  We 
simply need to collect the results for each of these test scenarios to verify 
the patch functionality and no regressions were introduced.


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