[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/690 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-14 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/690#issuecomment-131016340 @jayapalu Please see PR #693, I went ahead and included your work there. Please let me know if this is OK with you. I included the exact same commit from this PR

[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-13 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/690#issuecomment-130798498 Forgot to update this.. tested it and the rules are applied OK now. LGTM. I will add more firewall rules so the feature will work again. Let's also look

[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-13 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/690#issuecomment-130654437 @jayapalu happy you're helping out! If you found out more stuff already, feel free to post. Thanks! :-) --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-13 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/690#issuecomment-130602329 @remibergsma I thought you are only looking at the rules issue. You can look at the other issues in s2s vpn. You might have observed it but making it to yo

[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-13 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/690#issuecomment-130593053 Hi @jayapalu I also worked on this, and but even with the firewall rules applied, the functionality still doesn't work because some rules are missing. So, this m

[GitHub] cloudstack pull request: CLOUDSTACK-8710: Fixed applying iptables ...

2015-08-13 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/690 CLOUDSTACK-8710: Fixed applying iptables rules for s2s vpn @remibergsma @wilderrodrigues Moved applying iptables rules apply after vpn configuration so that vpn specific rules also get app