----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16279/#review30983 -----------------------------------------------------------
And this fix involved VPC, which I don't think would be contained in 4.3 branch. You can merge this with https://reviews.apache.org/r/16274/ I think. - Sheng Yang On Dec. 15, 2013, 11:57 p.m., Suresh Balineni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16279/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2013, 11:57 p.m.) > > > Review request for cloudstack. > > > Repository: cloudstack-git > > > Description > ------- > > Fixes for: > 1. contrail is updating Dao tables. DB updates should be inside Trasaction > start & commit calls. > 2. If contrail.properties file is not found set to default hostname & port id > where contrail server can be accessed from the plugin. > 3. Add JuniperContrailVPCRouter as SDN Provider in VpcManagerImpl. > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/16279/diff/ > > > Testing > ------- > > Tests are done. Verified. > > > File Attachments > ---------------- > > VPC Fix > > https://reviews.apache.org/media/uploaded/files/2013/12/15/0f239bf5-d9ba-4cc1-b232-ee4a6ef2e0e7__vpc-fix.patch > > > Thanks, > > Suresh Balineni > >