----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18552/ -----------------------------------------------------------
(Updated March 4, 2014, 10:11 p.m.) Review request for cloudstack. Changes ------- Hi Alena, Could you review the change in VpcManagerImpl.java? Repository: cloudstack-git Description ------- Internal LB support for Juniper contrail VPC implementation. - This implementation just extends the existing implementation of internal lb. - New element uses juniper contrail network offering so that nics are impelemented by contrail element. - LB VM deployment functionality is reused (new element is extended from existing Internal LB element implementation). Diffs ----- api/src/com/cloud/network/Network.java 6dc6752 api/src/org/apache/cloudstack/api/BaseCmd.java 0e83cee plugins/network-elements/juniper-contrail/pom.xml 8c6877d plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/spring-contrail-context.xml 99ab02e plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailInternalLbElementImpl.java PRE-CREATION plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java 01be7db server/src/com/cloud/network/vpc/VpcManagerImpl.java 2157eac Diff: https://reviews.apache.org/r/18552/diff/ Testing ------- Tested LB VM deployment. Traffic tests are done. Thanks, Suresh Balineni