rhtyd commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S 
VPN connections
URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-348693472
 
 
   @swill /cc @syed I fixed fixed the issue and got the marvin test to pass, 
see my notes on this PR. My fix is on this PR: 
https://github.com/apache/cloudstack/pull/2211.
   
   I've done three major things:
   - Increased retry count by 3, and ping the rightpeer:
   ```
   +        for i in xrange(3):
   +            result = CsHelper.execute('ipsec status vpn-%s | grep "%s"' % 
(rightpeer, peerlist.split(",", 1)[0]))
   +            if len(result) > 0:
   +                break
   +            time.sleep(1)
   +
   +        # With 'auto=route', connections are established with an attempt to 
communicate over the S2S VPN
   +        # Attempt to ping the other side to initialize the connection of 
the S2S VPN configuration
   +        CsHelper.execute("timeout 2 ping -c 2 %s" % (rightpeer))
   ```
   - In the tests, I've used `wait_until` to poll upto 60 seconds on the 
passive VPN connection (docs says it may take upto 30s for the VPN connection 
to transition into `Connected` state)
   - Fixed strongswan 5.5 compatible configs

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to