Message ID | 20240124142740.969176-5-mheib@redhat.com |
---|---|
State | Accepted |
Headers | show |
Series | OVN-IC: Add basic sequence number status support. | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | success | apply and check: success |
ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
ovsrobot/github-robot-_ovn-kubernetes | success | github build: passed |
On Wed, Jan 24, 2024 at 3:28 PM Mohammad Heib <mheib@redhat.com> wrote: > add unit test that check validate that sync command > sync ISB properly > > Signed-off-by: Mohammad Heib <mheib@redhat.com> > --- > tests/ovn-ic.at | 43 +++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 43 insertions(+) > > diff --git a/tests/ovn-ic.at b/tests/ovn-ic.at > index d4c436f84..535aba7da 100644 > --- a/tests/ovn-ic.at > +++ b/tests/ovn-ic.at > @@ -1274,3 +1274,46 @@ OVN_CLEANUP_IC([az1], [az2]) > > AT_CLEANUP > ]) > + > +OVN_FOR_EACH_NORTHD([ > +AT_SETUP([ovn-ic -- sync ISB status to INB]) > +ovn_init_ic_db > +net_add n1 > + > +ovn_start az1 > +sim_add gw-az1 > +as gw-az1 > + > +check ovs-vsctl add-br br-phys > +ovn_az_attach az1 n1 br-phys 192.168.1.1 > +check ovs-vsctl set open . external-ids:ovn-is-interconn=true > +as az1 > + > +# pause ovn-ic instance > +check ovn-appctl -t ic/ovn-ic pause > + > +# run sync command in the background this commands > +# supposed to stuck since ovn-ic is paused. > +ovn-ic-nbctl --wait=sb sync & > + > +OVS_WAIT_UNTIL([test $(ovn-ic-nbctl get ic_nb_global . nb_ic_cfg) -gt > $(ovn-ic-nbctl get ic_nb_global . sb_ic_cfg)]) > +AT_CHECK([ovn-ic-nbctl get ic_nb_global . nb_ic_cfg], [0], [dnl > +1 > +]) > +AT_CHECK([ovn-ic-nbctl get ic_nb_global . sb_ic_cfg], [0], [dnl > +0 > +]) > + > +# resume ovn-ic instance > +check ovn-appctl -t ic/ovn-ic resume > +OVS_WAIT_UNTIL([test $(ovn-ic-nbctl get ic_nb_global . nb_ic_cfg) -eq > $(ovn-ic-nbctl get ic_nb_global . sb_ic_cfg)]) > +AT_CHECK([ovn-ic-nbctl get ic_nb_global . nb_ic_cfg], [0], [dnl > +1 > +]) > +AT_CHECK([ovn-ic-nbctl get ic_nb_global . sb_ic_cfg], [0], [dnl > +1 > +]) > + > +OVN_CLEANUP_IC([az1]) > +AT_CLEANUP > +]) > -- > 2.34.3 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Looks good to me, thanks. Acked-by: Ales Musil <amusil@redhat.com>
diff --git a/tests/ovn-ic.at b/tests/ovn-ic.at index d4c436f84..535aba7da 100644 --- a/tests/ovn-ic.at +++ b/tests/ovn-ic.at @@ -1274,3 +1274,46 @@ OVN_CLEANUP_IC([az1], [az2]) AT_CLEANUP ]) + +OVN_FOR_EACH_NORTHD([ +AT_SETUP([ovn-ic -- sync ISB status to INB]) +ovn_init_ic_db +net_add n1 + +ovn_start az1 +sim_add gw-az1 +as gw-az1 + +check ovs-vsctl add-br br-phys +ovn_az_attach az1 n1 br-phys 192.168.1.1 +check ovs-vsctl set open . external-ids:ovn-is-interconn=true +as az1 + +# pause ovn-ic instance +check ovn-appctl -t ic/ovn-ic pause + +# run sync command in the background this commands +# supposed to stuck since ovn-ic is paused. +ovn-ic-nbctl --wait=sb sync & + +OVS_WAIT_UNTIL([test $(ovn-ic-nbctl get ic_nb_global . nb_ic_cfg) -gt $(ovn-ic-nbctl get ic_nb_global . sb_ic_cfg)]) +AT_CHECK([ovn-ic-nbctl get ic_nb_global . nb_ic_cfg], [0], [dnl +1 +]) +AT_CHECK([ovn-ic-nbctl get ic_nb_global . sb_ic_cfg], [0], [dnl +0 +]) + +# resume ovn-ic instance +check ovn-appctl -t ic/ovn-ic resume +OVS_WAIT_UNTIL([test $(ovn-ic-nbctl get ic_nb_global . nb_ic_cfg) -eq $(ovn-ic-nbctl get ic_nb_global . sb_ic_cfg)]) +AT_CHECK([ovn-ic-nbctl get ic_nb_global . nb_ic_cfg], [0], [dnl +1 +]) +AT_CHECK([ovn-ic-nbctl get ic_nb_global . sb_ic_cfg], [0], [dnl +1 +]) + +OVN_CLEANUP_IC([az1]) +AT_CLEANUP +])
add unit test that check validate that sync command sync ISB properly Signed-off-by: Mohammad Heib <mheib@redhat.com> --- tests/ovn-ic.at | 43 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+)