On Thu, 22 Feb 2018 18:07:55 +0100
Jakub Sitnicki <j...@redhat.com> wrote:

> On Wed, Feb 14, 2018 at 09:54 PM GMT, Ben Pfaff wrote:

[...]

> > +uint32_t
> > +sbrec_logical_flow_hash(const struct sbrec_logical_flow *lf)
> > +{
> > +    const struct sbrec_datapath_binding *ld = lf->logical_datapath;
> > +    if (!ld) {
> > +        return 0;
> > +    }
> > +
> > +    return ovn_logical_flow_hash(&ld->header_.uuid,
> > +                                 lf->table_id, lf->pipeline,
> > +                                 lf->priority, lf->match, lf->actions);
> > +}  
> 
> It looks like we should be hashing the logical datapath UUID here,
> instead of the UUID of the entry in Datapath_Binding. Right now, the
> hash computed from SB record will never match the hash computed from
> NBDB contents.
> 
> This leads to northd contantly dropping records from SBDB and inserting
> them again and again. I believe this is the root cause of problems that
> Mark and Numan are reporting.

Proposed fix now posted: https://patchwork.ozlabs.org/patch/876842/

Thanks,
Jakub
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to