----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16805/#review31838 -----------------------------------------------------------
Ship it! Applied to 4.3 branch. Thanks. - Sheng Yang On Jan. 14, 2014, 11:03 p.m., Suresh Balineni wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > > 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 > >
