Hi Sheng, This is a reminder. Can you please ship this?
Thanks, Suresh On Jan 11, 2014, at 4:13 PM, "Suresh Balineni" <[email protected]<mailto:[email protected]>> 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 <[email protected]<mailto:[email protected]>> Reply-To: Suresh Balineni <[email protected]<mailto:[email protected]>> Date: Saturday, January 11, 2014 4:08 PM To: Suresh Balineni <[email protected]<mailto:[email protected]>>, cloudstack <[email protected]<mailto:[email protected]>> 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/>
