GitHub user wilderrodrigues opened a pull request:

    https://github.com/apache/cloudstack/pull/923

    CLOUDSTACK-8934 - Default routes not configured for rVPC and RVR

    This PR fixes the default routes for redundant VPCs and isolated networks. 
New tests were introduced in order to make sure that the routers are working 
properly.
    
    During the tests, I found out that the Firewall Egress was not working 
properly when creating the network offering with default allow. The bug has 
been fixed and tests for redundant and non-redundant isolated networks were 
added.
    
    Test reports will follow in a separate comment.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ekholabs/cloudstack 
fix/default_routes_CLOUDSTACK-8934

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/923.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #923
    
----
commit 45642b83821ce0ecd6d4cddb76a77a2481e54d9a
Author: Wilder Rodrigues <wrodrig...@schubergphilis.com>
Date:   2015-10-08T12:19:26Z

    CLOUDSTACK-8934 - Add default gateway when the public interface is up again

commit 5d1cdc64409caed239f6814a8f73266c8e5eb6fe
Author: Wilder Rodrigues <wrodrig...@schubergphilis.com>
Date:   2015-10-09T11:31:32Z

    CLOUDSTACK-8934 - Adding tests to cover default routes on IsoNest and RVR 
nets

commit b4dc392bfdf4fb93e1652203b7d4027651cca5ac
Author: Wilder Rodrigues <wrodrig...@schubergphilis.com>
Date:   2015-10-09T12:32:35Z

    CLOUDSTACK-8934 - Fix default EGRESS rules for isolated networks
    
       - The default is Accept and will be changed based on the configuration 
of the offering.
    
    CLOUDSTACK-8934 - The default egress is set as Deny in the router.
    
       - We had to change it on the Java side in order to make the apply it 
once the default is defined as allowed on the net offering

commit cd8c3e97f594639292075d9a704dc7c382a103ca
Author: Wilder Rodrigues <wrodrig...@schubergphilis.com>
Date:   2015-10-10T10:02:47Z

    CLOUDSTACK-8934 - Fix the AclIP class to make it configure the default FW 
policy

----


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to