Re: [ovs-dev] [ovs-dev, v5] ovn-controller: Assign zone-id consistently

2016-02-23 Thread ramu
Thanks for your review! > > From this part of the patch, it looks like external-ids:zone-id is > accepted even if there are duplicates, or if the value is not valid. I > think that it should reject such cases: > > With that in mind, update_local_zone_ids() should also update >

Re: [ovs-dev] [ovs-dev, v5] ovn-controller: Assign zone-id consistently

2016-02-23 Thread Ben Pfaff
On Mon, Feb 15, 2016 at 03:46:38PM -0500, Ramu Ramamurthy wrote: > Currently, ovn-controller does not record the > lport->zoneid map, and so, after ovn-controller restart, > zone-ids may get set inconsistently on lports, resulting > in possible hits to already established connections. > > Set