Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/826#issuecomment-143727275 @remibergsma Rebased against the master. Repro steps are as follows: 1) Create two VMs in new isolated networks, testnet1 and testnet2. 2) Enable VPN in testnet1 and testnet2. 3) Add a VPN user in each network. 4) Notice that VPN users added. 5) Stop the VR for testnet2. 6) Add a VPN user in testnet1. 7) Notice that it fails with an error even though the testnet1 VR is running. The reason it is failing is that presently we there is only a check for the running VR's. If not running then error is thrown. Based on the present implementation of the VR rules, we should skip the STOPPED/STOPPING VR's as the rules get applied to them after they get restarted. To check rules are applied(VPN in this case), ssh to the NOW running VR and look into the /usr/share/ppp/chap-secrets file.
--- 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. ---