Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/908#issuecomment-145230344
  
    Hi @remibergsma @borisroman @DaanHoogland @miguelaferreira 
    
    I executed the RVR test here, see results below:
    
    ```
    Test redundant router internals ... === TestName: test_enableVPNOverRvR | 
Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 1 test in 466.394s
    
    OK
    /tmp//MarvinLogs/test_redundant_router_services_3QFJ63/results.txt (END)
    
    ```
    
    The test above does the following:
    
    1 listNetworks should show the created network in allocated state
    2 listRouters returns no running routers
    3 VMs should be deployed and in Running state
    4 should list 2 routers
    5 listPublicIpAddresses for networkid should show acquired IP addr
    6 listRemoteAccessVpns for the network associated should show VPN created
    7 listRemoteAccessVpns for the network associated should return empty 
response
    
    I don;t consider it enough, so I will write a test that check the 
conntrackd service in the routers as well.
    
    Should the PR wait until the test is ready or if I do it manually now (to 
close this blocker quicker) would that be enough?
    
    Cheers,
    WIlder


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