You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/09/27 14:11:40 UTC
[21/21] git commit: updated refs/heads/master to 3ded3e9
Merge pull request #887 from schubergphilis/vr_fixes_combined
[BLOCKER] Combined PRs that fix VR issuesTonight I worked with @wilderrodrigues to figure out what is wrong with the virtual router. As we couldn't test single PRs any more (because of other issues with them causing tests to fail) we added all VR related PRs in a separate branch and started testing from there.
We combined the following PRs into this PR:
#836 #851 #867 #870 #881 #882 #842
After that, one issue remains: the VPC does not get a default gateway. Which is strange, because we already solved it in PR #738. When I look back, it was fixed again in PR #784. It could very well be that either one fixed one specific case, but also breaking the other. We need to investigate this, and make sure there will be a fix that works both for VPCs and VRs.
When we manually add the default gateway on the VPC, most tests pass and also spinning up two VPCs with one tier each, having a VM and them using s2s to VPN them together works fine. See for more details the report Wilder sent earlier.
Tomorrow we'll try to figure out how to fix the default gateway and merge this. Then we should have a base to work from again. Any PR that fixes another blocker, should at least then be rebased against the fixed master so we can run the tests against the PR branch. I'm not saying everything is fixed, I'm just saying that we can spin up a cloud that has working VMs.
When, in the mean time, someone has the time to checkout this branch and make the default route work for both VPC and VR that would be awesome. After that we should double check and verify the test results.
Pinging @karuturi to let her know the current status.
Regards,
Wilder / Remi
* pr/887:
Fixing the index out of bounds error in the check_if_link_up() function
small cleanups
Fixing the defaut route for VPC routers
Formatting the get_gateway() method in the CsDatabag.py file
Fixing the dhcpsrvr iptables file
Formatting the router_proxy.sh script
CLOUDSTACK-8881: Fixed Static and PF configuration issue
CLOUDSTACK-8905: Fixed hooking egress rules
CLOUDSTACK-8891: Fixed default iptables rules on VR for guest traffic
Configured dnsmasq to listen on all interfaces so that vpn client gets dns
CLOUDSTACK-8864: Not able to add TCP port forwarding rule in VPN for specific ports
CLOUDSTACK-8863: VM doesn't reconnect to internet post VR RESTART/STOP-START/RECREATE
CLOUDSTACK-8843: Fixed issue in default iptables rules on shared network VR
Signed-off-by: Remi Bergsma <gi...@remi.nl>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/3ded3e90
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/3ded3e90
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/3ded3e90
Branch: refs/heads/master
Commit: 3ded3e90007d08fa98465f2b8c68b7fb075557c0
Parents: 415631a 09e05f2
Author: Remi Bergsma <gi...@remi.nl>
Authored: Sun Sep 27 14:09:48 2015 +0200
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Sun Sep 27 14:09:48 2015 +0200
----------------------------------------------------------------------
scripts/network/domr/router_proxy.sh | 11 ---
.../network/firewall/FirewallManagerImpl.java | 3 +-
.../network/firewall/FirewallManagerTest.java | 83 +++++++++++++++++---
.../patches/debian/config/etc/dnsmasq.conf.tmpl | 4 +-
.../debian/config/etc/init.d/cloud-early-config | 2 -
.../config/etc/iptables/iptables-dhcpsrvr | 58 ++++++++++++++
.../debian/config/opt/cloud/bin/configure.py | 40 +++++++++-
.../debian/config/opt/cloud/bin/cs/CsAddress.py | 34 ++++----
.../debian/config/opt/cloud/bin/cs/CsDatabag.py | 1 +
.../config/opt/cloud/bin/cs/CsNetfilter.py | 6 +-
.../debian/config/opt/cloud/bin/vr_cfg.sh | 11 +++
11 files changed, 210 insertions(+), 43 deletions(-)
----------------------------------------------------------------------