On Mon, Feb 26, 2018 at 11:47:08AM +0100, Jakub Sitnicki wrote:
> On Fri, Feb 23, 2018 at 09:05 PM GMT, Ben Pfaff wrote:
> > On Thu, Feb 22, 2018 at 10:54:05PM +0100, Jakub Sitnicki wrote:
> >> Use the logical switch/router UUID as hash input, instead of the UUID of
> >> the Datapath_Binding row, w
On Fri, Feb 23, 2018 at 09:05 PM GMT, Ben Pfaff wrote:
> On Thu, Feb 22, 2018 at 10:54:05PM +0100, Jakub Sitnicki wrote:
>> Use the logical switch/router UUID as hash input, instead of the UUID of
>> the Datapath_Binding row, when calculating the hash value for lflows in
>> the SBDB.
>>
>> Otherwis
On Thu, Feb 22, 2018 at 10:54:05PM +0100, Jakub Sitnicki wrote:
> Use the logical switch/router UUID as hash input, instead of the UUID of
> the Datapath_Binding row, when calculating the hash value for lflows in
> the SBDB.
>
> Otherwise the hash value will never match the one computed from NBDB
Jakub and I discussed this patch offline and I am now on board with this
patch.
Acked-By: Mark Michelson
Tested-By: Mark Michelson
On 02/23/2018 08:42 AM, Mark Michelson wrote:
Hi Jakub,
This patch fixes the issue. However, I think the resulting code can lead
to some confusion. The name of
Hi Jakub,
This patch fixes the issue. However, I think the resulting code can lead
to some confusion. The name of the first parameter given to
ovn_logical_flow_hash() is called "logical_datapath". The result of your
patch is that the actual first parameter passed to that function is
either a
On Fri, Feb 23, 2018 at 3:24 AM, Jakub Sitnicki wrote:
> Use the logical switch/router UUID as hash input, instead of the UUID of
> the Datapath_Binding row, when calculating the hash value for lflows in
> the SBDB.
>
> Otherwise the hash value will never match the one computed from NBDB
> conten
Use the logical switch/router UUID as hash input, instead of the UUID of
the Datapath_Binding row, when calculating the hash value for lflows in
the SBDB.
Otherwise the hash value will never match the one computed from NBDB
contents, which will force ovn-northd to constantly drop and attempt to
re