Github user miguelaferreira commented on the pull request:

    https://github.com/apache/cloudstack/pull/1094#issuecomment-167105359
  
    I've tested this PR with the additional 
[commits](https://github.com/nvazquez/cloudstack/pull/1) that I've requested to 
be merged here.
    
    My environment:
    * Management Server
    * 2 KVM hypervisors
    * 3 NSX controllers
    * 1 NSX Service Node
    * 2 NSX Gateways
    * 1 NSX Manager
    
    NSX setup:
    * 1 transport zone (with service node, hypervisors and gateways configured 
in it)
    * 1 L2 Gateway service (connected to a NSX Gateway VM)
    * 1 L3 Gateway service (connected to a NSX Gateway VM)
    * 1 LRouter connected to the L# Gateway service
    
    Same [marvin 
configuration](https://github.com/miguelaferreira/MCT-shared/blob/9b64fd70a765e35778b1cc768edc03dbb3924654/marvin/mct-zone1-kvm1-kvm2-NVP.cfg)
 used for deploying DC and running tests.
    
    Tests:
    ```
    nosetests --with-marvin 
--marvin-config=/data/shared/marvin/mct-zone1-kvm1-kvm2-NVP-2.cfg -s -a 
tags=advanced,required_hardware=true 
test/integration/plugins/test_nicira_controller.py
    (...)
    test_01_nicira_controller (test_nicira_controller.TestNiciraContoller) ... 
=== TestName: test_01_nicira_controller | Status : SUCCESS ===
    ok
    Nicira clusters will redirect clients (in this case ACS) to the master 
node. ... === TestName: test_02_nicira_controller_redirect | Status : SUCCESS 
===
    ok
    test_03_nicira_tunnel_guest_network 
(test_nicira_controller.TestNiciraContoller) ... === TestName: 
test_03_nicira_tunnel_guest_network | Status : SUCCESS ===
    ok
    Shared Networks Support ... === TestName: 
test_04_nicira_shared_networks_numerical_vlanid | Status : SUCCESS ===
    ok
    Shared Networks Support ... === TestName: 
test_05_nicira_shared_networks_lrouter_uuid_vlan_id | Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 5 tests in 1961.460s
    
    OK
    ```
    
    @nvazquez if you can reproduce the same test results in your environment, 
then I think we have tested this PR thoroughly.


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