cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wilderrodrigues <...@git.apache.org>
Subject [GitHub] cloudstack pull request: CLOUDSTACK-9106 - As a Developer I want t...
Date Mon, 07 Dec 2015 13:43:10 GMT
GitHub user wilderrodrigues opened a pull request:

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

    CLOUDSTACK-9106 - As a Developer I want the Redundant VPC private gateway feature fixed

    This PR contains the same fixes from PR #1179, which was created against the master branch.
    
    In addition, the points mentioned by @DaanHoogland were handled in this new PR:
    
    * Made the code more consistent
      - result = result && methodCall(), instead of throwing exceptions in some places
or not checking 2 consecutive returns - in case of rVPC.
    * Added an unit test to cover changes in the VpcRouterElementImpl.applyVpnUsers() method.
The method returns an array of String, so I had to make sure it would contain the users from
2 consecutive calls. There are 2 tests to cover negative scenarios.

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

    $ git pull https://github.com/ekholabs/cloudstack fix/4.6-rvpc-pvtgw-CLOUDSTACK-9106

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

    https://github.com/apache/cloudstack/pull/1184.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 #1184
    
----
commit a168a69c3cdf023a296ad9a8793e3ab4d7e47cee
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-04T17:41:03Z

    CLOUDSTACK-9106 - Reduces the amount of iterations through the routers of a VPC
    
       - It was causing problems because Nics were expected to be plugged before they actually
exist. Only in rVPC cases.
       - Applies ACL items to routers only after the Pvt GW is setup.

commit 24dafc2cc242b69e986d83a94baae30af368b132
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-05T13:03:50Z

    CLOUDSTACK-9106 - Refactor the createPrivateNicProfileForGateway() method
    
        - Use the router to retrieve the instance ID
        - Check if the VPC is redundant in order to reuse the private gateway address.
        - Brings the private gateways interfaces up.

commit 1738ce174005096f4106673bb5b35cd1351a5f56
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-05T16:33:55Z

    CLOUDSTACK-9106 - Enables private gateway tests on Redundant VPCs

commit 13eb78938820489efc0251cf85ed4e7fd01f685d
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-07T12:40:12Z

    CLOUDSTACK-9106 - Makes the router commands call more consistent.
    
       - Checks the result of a call against the previous result. Either both are true or
the method returns false.
       - Do not thrown exceptions because some calls are not handling/rethrowing them. It
would cause runtime problems.
       - When doing a list.addAll(Arrays.asList(String[]{}) will cause problems when trying
to cast the list.toArray() into an aray of String
         It would only work if instead of calling addAll() I would pass it straight into the
constructor:
         e.g. List<String> l = new ArrayList(Arrays.asList(new String[]{});
              Stirng [] s = (String[]) l.toArray();
         But I did not like that implementation because it would require 2 arrays of string
and combine them at the end.

commit 5973f4ea77ed3328e74c79069dcf5b1eff2f049f
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-07T12:42:29Z

    CLOUDSTACK-9106 - Adds a test to cover the changes in the applyVpnUsers() method
    
       - Changed the NetworkTopologyContext class just to make the private member accessible
from the test
       - Added a test class to cover the positive scenario of the VpcVirtualRouterElementTest.applyVpnUsers()
method.
       - Covering when there is either no VPC or no routers.

commit 14db2d3dc4f8afefaa48a52c70d8141b8b4222ad
Author: Wilder Rodrigues <wrodrigues@schubergphilis.com>
Date:   2015-12-07T12:43:46Z

    CLOUDSTACK-9106 - Makes Enum name compliant with Java code conventions.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message