----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22093/#review44480 -----------------------------------------------------------
Another issue might be that new entries (we see this on ACS 4.3 with XenServer) are created without vlan:// The name of the field is vlan_id and not BroadcastURI so keeping it an integer only would make more sense to me. And then use null for untagged vlans. If we would really want a bURI then we should also rename the field. - Joris van Lieshout On May 30, 2014, 9 p.m., daan Hoogland wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22093/ > ----------------------------------------------------------- > > (Updated May 30, 2014, 9 p.m.) > > > Review request for cloudstack and Marcus Sorensen. > > > Repository: cloudstack-git > > > Description > ------- > > VPC's VR missing public NIC eth1 as reported by Andrija Panic. fix by > updating on upgrade > > > Diffs > ----- > > engine/schema/src/com/cloud/upgrade/dao/Upgrade421to430.java 7e26132 > > Diff: https://reviews.apache.org/r/22093/diff/ > > > Testing > ------- > > > Thanks, > > daan Hoogland > >
