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