[ovs-dev] [PATCH v4 3/3] ovsdb-idl: Break into two layers.
Ben Pfaff
blp at ovn.org
Fri Jan 22 03:08:38 UTC 2021
On Fri, Jan 22, 2021 at 12:50:38AM +0100, Ilya Maximets wrote:
> On 1/22/21 12:42 AM, Ben Pfaff wrote:
> > On Wed, Jan 13, 2021 at 05:32:52PM +0100, Dumitru Ceara wrote:
> >> On 1/13/21 2:56 AM, Ilya Maximets wrote:
> >>> On the side note: don't we need to increase 'cond_seqno' here since
> >>> we're practically dropping one of the condition requests? User might
> >>> wait for this sequence number, but 'req_cond' will be cleared and
> >>> never be acked.
> >>
> >> I think you're right, we should be increasing 'cond_seqno' to match the
> >> logic in ovsdb_cs_db_set_condition() (or ovsdb_idl_db_set_condition() in
> >> the old code).
> >>
> >> I can take care of this as a follow up patch as it seems to be
> >> pre-existing issue of the IDL code.
> >
> > I found that any minor change to the IDL behavior in this area tended to
> > break ovn-controller, which seems to have really brittle dependencies on
> > exactly how the IDL behaves.
>
> Yep. I think, narrowing down direct idl usage from the ovn-controller side
> is one of the main ToDo items for the refactoring we talked about on some
> of the previous OVN meetings.
>
> >
> > Thanks for the reviews. I got the impression from these and from the
> > discussion on IRC this morning that it was OK to push this to master, so
> > I did.
> >
>
> Sure, thanks!
>
> I think we also agreed to apply the series to branch-2.15 (it was already
> created last week). Will you do that?
I did that also.
More information about the dev
mailing list