----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9651/#review17399 -----------------------------------------------------------
Ship it! Pushed to 4.1 branch. Please update the bug to set to resolved. commit 17e3313e694940efb97eb4aadec336869c24022d Author: Venkata Swamybabu Budumuru <venkataswamybabu.budum...@citrix.com> Date: Wed Feb 27 13:37:41 2013 -0500 Accounts can hold multiple vnets. Same vnets can exist across multiple pnets. It is sufficient to constrain a single vnet in a given pnet within a datacenter. `i_op_dc_vnet_alloc__vnet__data_center_id` covers all use cases for the table. Having `i_op_dc_vnet_alloc__vnet__data_center_id__account_id` will restrict same account holding the same vnet across different physical networks. Signed-off-by: Venkata Swamybabu Budumuru <venkataswamybabu.budum...@citrix.com> - Chip Childers On March 5, 2013, 1:31 p.m., venkata swamy babu budumuru wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9651/ > ----------------------------------------------------------- > > (Updated March 5, 2013, 1:31 p.m.) > > > Review request for cloudstack, Chip Childers and Marcus Sorensen. > > > Description > ------- > > to repro > > (1) Have an advanced zone with physical networks say Phynet1: 900-905 > Phynet2: 905-909 > (2) create a Network in physical network1 (id : 200) and deploy a vm (for > ex:- network created is using vlan 905 ) > (3) create another network in physical network2 (id : 201) and deploy a vm > > Observations : > > (i) It tried to pick up vlan 905 from physical network 2 (id : 201) and it > failed with the following error. > > Caused by: > com.mysql.jdbc.exceptions.jdbc4.MySQLIntegrityConstraintViolationException: > Duplicate entry '905-1-3' for key > 'i_op_dc_vnet_alloc__vnet__data_center_id__account_id' > > Attached patch fixes the issue. > > > This addresses bug CLOUDSTACK-1429. > > > Diffs > ----- > > setup/db/db/schema-40to410.sql 59b2a7d > > Diff: https://reviews.apache.org/r/9651/diff/ > > > Testing > ------- > > Retried the scenario with the dropped index and it works successfully. > > > Thanks, > > venkata swamy babu budumuru > >