Message ID | 20240404171302.54562-1-ihrachys@redhat.com |
---|---|
State | Not Applicable |
Headers | show |
Series | [ovs-dev] Make tunnel ids exhaustion test case trigger the problem. | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | fail | apply and check: fail |
ovsrobot/intel-ovs-compilation | fail | test: fail |
Sorry, missing 'ovn' in the subject. Resent to ovn queue. On Thu, Apr 4, 2024 at 1:13 PM Ihar Hrachyshka <ihrachys@redhat.com> wrote: > The original version of the scenario passed with or without the fix. > > Fixes: a1f165a7b807 ("northd: fix infinite loop in ovn_allocate_tnlid()") > Signed-off-by: Ihar Hrachyshka <ihrachys@redhat.com> > --- > tests/ovn-northd.at | 17 +++++++++++------ > 1 file changed, 11 insertions(+), 6 deletions(-) > > diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at > index fc2c972a4..e8ea8b050 100644 > --- a/tests/ovn-northd.at > +++ b/tests/ovn-northd.at > @@ -2826,11 +2826,6 @@ OVN_FOR_EACH_NORTHD_NO_HV([ > AT_SETUP([check tunnel ids exhaustion]) > ovn_start > > -# Create a fake chassis with vxlan encap to lower MAX DP tunnel key to > 2^12 > -ovn-sbctl \ > - --id=@e create encap chassis_name=hv1 ip="192.168.0.1" type="vxlan" \ > - -- --id=@c create chassis name=hv1 encaps=@e > - > cmd="ovn-nbctl --wait=sb" > > for i in {1..4097}; do > @@ -2840,7 +2835,17 @@ done > eval $cmd > > check_row_count nb:Logical_Switch 4097 > -wait_row_count sb:Datapath_Binding 4095 > +wait_row_count sb:Datapath_Binding 4097 > + > +# Now create a fake chassis with vxlan encap to lower MAX DP tunnel key > to 2^12 > +ovn-sbctl \ > + --id=@e create encap chassis_name=hv1 ip="192.168.0.1" type="vxlan" \ > + -- --id=@c create chassis name=hv1 encaps=@e > + > +ovn-nbctl --wait=sb ls-add lsw-exhausted > + > +check_row_count nb:Logical_Switch 4098 > +wait_row_count sb:Datapath_Binding 4097 > > OVS_WAIT_UNTIL([grep "all datapath tunnel ids exhausted" > northd/ovn-northd.log]) > > -- > 2.41.0 > >
diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at index fc2c972a4..e8ea8b050 100644 --- a/tests/ovn-northd.at +++ b/tests/ovn-northd.at @@ -2826,11 +2826,6 @@ OVN_FOR_EACH_NORTHD_NO_HV([ AT_SETUP([check tunnel ids exhaustion]) ovn_start -# Create a fake chassis with vxlan encap to lower MAX DP tunnel key to 2^12 -ovn-sbctl \ - --id=@e create encap chassis_name=hv1 ip="192.168.0.1" type="vxlan" \ - -- --id=@c create chassis name=hv1 encaps=@e - cmd="ovn-nbctl --wait=sb" for i in {1..4097}; do @@ -2840,7 +2835,17 @@ done eval $cmd check_row_count nb:Logical_Switch 4097 -wait_row_count sb:Datapath_Binding 4095 +wait_row_count sb:Datapath_Binding 4097 + +# Now create a fake chassis with vxlan encap to lower MAX DP tunnel key to 2^12 +ovn-sbctl \ + --id=@e create encap chassis_name=hv1 ip="192.168.0.1" type="vxlan" \ + -- --id=@c create chassis name=hv1 encaps=@e + +ovn-nbctl --wait=sb ls-add lsw-exhausted + +check_row_count nb:Logical_Switch 4098 +wait_row_count sb:Datapath_Binding 4097 OVS_WAIT_UNTIL([grep "all datapath tunnel ids exhausted" northd/ovn-northd.log])
The original version of the scenario passed with or without the fix. Fixes: a1f165a7b807 ("northd: fix infinite loop in ovn_allocate_tnlid()") Signed-off-by: Ihar Hrachyshka <ihrachys@redhat.com> --- tests/ovn-northd.at | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-)