----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16805/ -----------------------------------------------------------
(Updated Jan. 14, 2014, 11:03 p.m.) Review request for cloudstack. Changes ------- Hi Sheng, I updated the fix without db transaction wrappers. This is a 4.3 version of patch. Please review and ship it. Thanks, Suresh 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. Diffs (updated) ----- 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 b7c0023 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java d5a8722 Diff: https://reviews.apache.org/r/16805/diff/ Testing ------- verified locally. Thanks, Suresh Balineni
