Message ID | 20240109132854.669822-1-mheib@redhat.com |
---|---|
Headers | show |
Series | OVN-IC: add basic sequence number status support | expand |
On Tue, Jan 9, 2024 at 2:29 PM Mohammad Heib <mheib@redhat.com> wrote: > Currently, OVN-IC doesn't support a way to tell the end-user when their > changes > to the IC-NB database have propagated successfully to the IC-SB Database. > > This patch series adds basic support for the sequence number status > protocol > that is implemented on the native OVN, with this patch series the end user > now can wait for their changes in the IC-NB DB to take effect by executing > the > 'sync' command after applying any changes to the IC-NB DB, for example, if > the > end-user has created a transit switch in the IC-NB global DB and want to > make > sure that the IC-SB create a DP binding for this ts-switch the user now > can use > the 'sync' command as following: > $ ovn-ic-nbctl ts-add ts1 > $ ovn-ic-nbctl --wait=sb sync > > The second command will wait until all the ovn-ic instances see the new > changes > and update their own local dbs and the global IC-SB db. > > v3 -> v4 > -------- > * Addressed review comments from Ales to add check for the > overflow cases. > > v2 -> v3 > -------- > * Rebase over main. > * Addressed review comments from Mark and Ales. > > > Mohammad Heib (4): > OVN-IC: interconnect DBs add basic Information Flow columns > ovn-ic: implement basic INB change handling status > OVN-IC: Make it possible for CMS to detect when the ISB is up-to-date. > ic/tests: add unit test for ic sync command > > NEWS | 8 +++ > ic/ovn-ic.c | 96 ++++++++++++++++++++++++++++++++++-- > ovn-ic-nb.ovsschema | 6 ++- > ovn-ic-nb.xml | 17 +++++++ > ovn-ic-sb.ovsschema | 8 +-- > ovn-ic-sb.xml | 21 ++++++++ > tests/ovn-ic.at | 47 ++++++++++++++++++ > utilities/ovn-ic-nbctl.8.xml | 49 ++++++++++++++++++ > utilities/ovn-ic-nbctl.c | 89 ++++++++++++++++++++++++++++++++- > 9 files changed, 329 insertions(+), 12 deletions(-) > > -- > 2.34.3 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Hi Mohammad, thank you for the series. This suggestion is out of scope of this series, but it would be great to use the new sync across all ic tests that we currently have to further stabilize them. WDYT? Thanks, Ales
Hi Ales, Yes sure that's what I'm planning to do next, i first wanted to make those patches accepted upstream and then update the test case to use the sync command, i guess it will be easier to review if send the test cases change as a separate patch, what do you think? On Fri, Jan 12, 2024 at 11:14 AM Ales Musil <amusil@redhat.com> wrote: > > > On Tue, Jan 9, 2024 at 2:29 PM Mohammad Heib <mheib@redhat.com> wrote: > >> Currently, OVN-IC doesn't support a way to tell the end-user when their >> changes >> to the IC-NB database have propagated successfully to the IC-SB Database. >> >> This patch series adds basic support for the sequence number status >> protocol >> that is implemented on the native OVN, with this patch series the end user >> now can wait for their changes in the IC-NB DB to take effect by >> executing the >> 'sync' command after applying any changes to the IC-NB DB, for example, >> if the >> end-user has created a transit switch in the IC-NB global DB and want to >> make >> sure that the IC-SB create a DP binding for this ts-switch the user now >> can use >> the 'sync' command as following: >> $ ovn-ic-nbctl ts-add ts1 >> $ ovn-ic-nbctl --wait=sb sync >> >> The second command will wait until all the ovn-ic instances see the new >> changes >> and update their own local dbs and the global IC-SB db. >> >> v3 -> v4 >> -------- >> * Addressed review comments from Ales to add check for the >> overflow cases. >> >> v2 -> v3 >> -------- >> * Rebase over main. >> * Addressed review comments from Mark and Ales. >> >> >> Mohammad Heib (4): >> OVN-IC: interconnect DBs add basic Information Flow columns >> ovn-ic: implement basic INB change handling status >> OVN-IC: Make it possible for CMS to detect when the ISB is up-to-date. >> ic/tests: add unit test for ic sync command >> >> NEWS | 8 +++ >> ic/ovn-ic.c | 96 ++++++++++++++++++++++++++++++++++-- >> ovn-ic-nb.ovsschema | 6 ++- >> ovn-ic-nb.xml | 17 +++++++ >> ovn-ic-sb.ovsschema | 8 +-- >> ovn-ic-sb.xml | 21 ++++++++ >> tests/ovn-ic.at | 47 ++++++++++++++++++ >> utilities/ovn-ic-nbctl.8.xml | 49 ++++++++++++++++++ >> utilities/ovn-ic-nbctl.c | 89 ++++++++++++++++++++++++++++++++- >> 9 files changed, 329 insertions(+), 12 deletions(-) >> >> -- >> 2.34.3 >> >> _______________________________________________ >> dev mailing list >> dev@openvswitch.org >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> >> > Hi Mohammad, > > thank you for the series. This suggestion is out of scope of this series, > but it would be great to use the new sync across all ic tests that we > currently have to further stabilize them. WDYT? > > Thanks, > Ales > > > -- > > Ales Musil > > Senior Software Engineer - OVN Core > > Red Hat EMEA <https://www.redhat.com> > > amusil@redhat.com > <https://red.ht/sig> >
On Wed, Jan 24, 2024 at 9:44 AM Mohammad Heib <mheib@redhat.com> wrote: > Hi Ales, > > Yes sure that's what I'm planning to do next, i first wanted to make those > patches accepted upstream and then update the test case to use > the sync command, i guess it will be easier to review if send the test > cases change as a separate patch, what do you think? > Separate series/patch is completely fine once this series is accepted. Thanks, Ales > > On Fri, Jan 12, 2024 at 11:14 AM Ales Musil <amusil@redhat.com> wrote: > >> >> >> On Tue, Jan 9, 2024 at 2:29 PM Mohammad Heib <mheib@redhat.com> wrote: >> >>> Currently, OVN-IC doesn't support a way to tell the end-user when their >>> changes >>> to the IC-NB database have propagated successfully to the IC-SB Database. >>> >>> This patch series adds basic support for the sequence number status >>> protocol >>> that is implemented on the native OVN, with this patch series the end >>> user >>> now can wait for their changes in the IC-NB DB to take effect by >>> executing the >>> 'sync' command after applying any changes to the IC-NB DB, for example, >>> if the >>> end-user has created a transit switch in the IC-NB global DB and want to >>> make >>> sure that the IC-SB create a DP binding for this ts-switch the user now >>> can use >>> the 'sync' command as following: >>> $ ovn-ic-nbctl ts-add ts1 >>> $ ovn-ic-nbctl --wait=sb sync >>> >>> The second command will wait until all the ovn-ic instances see the new >>> changes >>> and update their own local dbs and the global IC-SB db. >>> >>> v3 -> v4 >>> -------- >>> * Addressed review comments from Ales to add check for the >>> overflow cases. >>> >>> v2 -> v3 >>> -------- >>> * Rebase over main. >>> * Addressed review comments from Mark and Ales. >>> >>> >>> Mohammad Heib (4): >>> OVN-IC: interconnect DBs add basic Information Flow columns >>> ovn-ic: implement basic INB change handling status >>> OVN-IC: Make it possible for CMS to detect when the ISB is up-to-date. >>> ic/tests: add unit test for ic sync command >>> >>> NEWS | 8 +++ >>> ic/ovn-ic.c | 96 ++++++++++++++++++++++++++++++++++-- >>> ovn-ic-nb.ovsschema | 6 ++- >>> ovn-ic-nb.xml | 17 +++++++ >>> ovn-ic-sb.ovsschema | 8 +-- >>> ovn-ic-sb.xml | 21 ++++++++ >>> tests/ovn-ic.at | 47 ++++++++++++++++++ >>> utilities/ovn-ic-nbctl.8.xml | 49 ++++++++++++++++++ >>> utilities/ovn-ic-nbctl.c | 89 ++++++++++++++++++++++++++++++++- >>> 9 files changed, 329 insertions(+), 12 deletions(-) >>> >>> -- >>> 2.34.3 >>> >>> _______________________________________________ >>> dev mailing list >>> dev@openvswitch.org >>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >>> >>> >> Hi Mohammad, >> >> thank you for the series. This suggestion is out of scope of this series, >> but it would be great to use the new sync across all ic tests that we >> currently have to further stabilize them. WDYT? >> >> Thanks, >> Ales >> >> >> -- >> >> Ales Musil >> >> Senior Software Engineer - OVN Core >> >> Red Hat EMEA <https://www.redhat.com> >> >> amusil@redhat.com >> <https://red.ht/sig> >> >