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/12/03 20:43:38 UTC

[09/11] git commit: updated refs/heads/master to 7e902cd

Merge pull request #1151 from ekholabs/fix/private_gw_rVPC-CLOUDSTACK-9075

[4.6] CLOUDSTACK-9075 - As a Developer I want the Private GW feature fixed on single VPCsThis PR fixes the issue we faced with Private Gateways on single VPC when using ACS 4.6.0 and onwards.

The root cause: during the VR refactor, the static routes configuration was left unimplemented.

This PR also improves the existing Replace ACL test and adds a new test, that cover the Private Gateway in a more complete way.

The new test does the following:

1. Create 2 VPCs
2. Create 2 Tiers - 1 per VPC
3. Deploy 2 VMs - 1 per Tier
4. Acquire 2 pub IPs - 1 per VPC
5. Create 2 PF rules - 1 per pub IP
6. Create 2 ACLs + rules - 1 per VPC
7. Assign new ACLs to Tiers
8. Create 2 Private GWs - 1 per VPC
9. Replace the Pvt GWs ACLs
10. Create 2 Static routes - 1 per Pvt GW
11. SSH into VM1 (VPC1) and from there ping VM2 (VPC2)

There is also a test for Private Gateways on Redundant VPCs. But I found out that the feature is broken in when used with rVPCs. It will be addressed in a separate Issue/PR.

I'm running the tests. Will post results as soon as they are ready.

* pr/1151:
  CLOUDSTACK-9075 - Uses the same vlan since it should have been already released
  CLOUDSTACK-9075 - Adds VPC static routes test
  CLOUDSTACK-9075 - Covers Private GW ACL with Redundant VPCs
  CLOUDSTACK-9075 - Add method to get list of Physical Networks per zone

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/519ce868
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/519ce868
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/519ce868

Branch: refs/heads/master
Commit: 519ce868af097d842e3b91472c8ce07b000b2e16
Parents: 5b7d935 6d9a3d8
Author: Remi Bergsma <gi...@remi.nl>
Authored: Thu Dec 3 19:56:08 2015 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Thu Dec 3 19:56:09 2015 +0100

----------------------------------------------------------------------
 .../debian/config/opt/cloud/bin/configure.py    |  29 +-
 .../config/opt/cloud/bin/cs_staticroutes.py     |  30 ++
 .../debian/config/opt/cloud/bin/merge.py        |   6 +
 test/integration/smoke/test_privategw_acl.py    | 382 ++++++++++++++++---
 tools/marvin/marvin/lib/common.py               |  11 +
 5 files changed, 394 insertions(+), 64 deletions(-)
----------------------------------------------------------------------