Hi Sheng, This is a reminder. Can you please ship this?
Thanks, Suresh On Jan 11, 2014, at 4:13 PM, "Suresh Balineni" <sbalin...@service.juniper.net<mailto:sbalin...@service.juniper.net>> wrote: Hi Sheng, This is another 4.3 review request. Code changes are minimal. Can you please review and the update the repository? Thanks, Suresh From: Suresh Balineni <sbalin...@juniper.net<mailto:sbalin...@juniper.net>> Reply-To: Suresh Balineni <sbalin...@service.juniper.net<mailto:sbalin...@service.juniper.net>> Date: Saturday, January 11, 2014 4:08 PM To: Suresh Balineni <sbalin...@service.juniper.net<mailto:sbalin...@service.juniper.net>>, cloudstack <dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>> Subject: Review Request 16805: Contrail plugin fixes - db update, policy port end fix This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16805/ Review request for cloudstack. By Suresh Balineni. Repository: cloudstack-git Description Contrail plugin fixes. - db updates must be done inside a transaction. - network policy port end type error. - default host contrail vnc is set to local host. This is considered if config is not present. Testing verified locally. Diffs * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java (18e8bde) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java (eea5026) * plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java (d5a8722) View Diff<https://reviews.apache.org/r/16805/diff/>
