[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-04-09 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-816655461


   @dhansenippathways can you help test #4595 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-04-09 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-816654906


   @rhtyd @soreana @ggoodrich-ipp, I backported to 4.15 branch in #4904
   
   please have a look and a spin. cc @borisstoyanov @weizhouapache 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-03-02 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-788748027


   @weizhouapache Centos7 does not support live migration like ubuntu does due 
to the default installation of qemu. I think this default must have changesd 
recently but it was never stable. My fix would mean checking the host os and 
skipping the test if it is `Centos`. I would merge this with the live migration 
tests failing on centos. I'll start an ubuntu test.
   @blueorangutan test centos7 kvm-ubuntu



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-03-02 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-788706978


   not sure about the static rules failure but the life cycle failures are due 
to centos7 limitation which are addressed in #4727 amongst many other things, 
This can be easily ported to a separate PR if we need to; 
https://github.com/apache/cloudstack/pull/4727/commits/8eec8c6cf6a6784ee7bc24a50114414d33b8b698



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-03-01 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-788021342


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-02-23 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-784047226


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-02-23 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-784016886


   don't worry @dhansenippathways , these sometimes happen because we are to 
heavy for travis. I restarted those and we test in other ways as well.
   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2021-01-29 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-769760426


   @ggoodrich-ipp all the ui conflicts are to files that should now moved to 
ui/legacy. However I connot see them on github. (cc @rhtyd ?)



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-11-10 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-724933097


   so do we move this to 4.16, @rhtyd ?



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-10-22 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-714368059


   @rhtyd @wido @weizhouapache do we merge this?



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-10-21 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-713353358


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-10-20 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-713051623


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-10-20 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-712778564


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-09-25 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-698768823


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-09-25 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-698768823


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-09-23 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-697889908


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-09-21 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-696034244


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-09-21 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-696034244


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-08-10 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-671235144


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [cloudstack] DaanHoogland commented on pull request #4137: Adding VPN options for IKE version and IKE split connections.

2020-07-29 Thread GitBox


DaanHoogland commented on pull request #4137:
URL: https://github.com/apache/cloudstack/pull/4137#issuecomment-665083905


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org