Skip to content

Commit

Permalink
Missed during test simplification.
Browse files Browse the repository at this point in the history
the testing was changed to remove the hardcoded table numbers from
flows. Two references to table=28 were missed, this patch corrects that
and adds the correct variable name to the flow.

Fixes: f614335 ("tests: Remove table numbers from "action parsing".")
Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com>
Acked-by: Ales Musil <amusil@redhat.com>
Signed-off-by: Mark Michelson <mmichels@redhat.com>
  • Loading branch information
JacobTanenbaum authored and putnopvut committed Apr 4, 2024
1 parent 5dd5737 commit de76bed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/ovn.at
Original file line number Diff line number Diff line change
Expand Up @@ -22073,7 +22073,7 @@ check_row_count Port_Binding 1 logical_port=sw0-vir virtual_parent=sw0-p1
wait_for_ports_up sw0-vir
check ovn-nbctl --wait=hv sync
AT_CHECK([test 2 = `cat hv1/ovn-controller.log | grep "pinctrl received packet-in" | \
grep opcode=BIND_VPORT | grep OF_Table_ID=28 | wc -l`])
grep opcode=BIND_VPORT | grep OF_Table_ID=$(ovn-debug lflow-stage-to-oftable ls_in_arp_rsp) | wc -l`])

wait_row_count Port_Binding 1 logical_port=sw0-vir6 chassis=$hv1_ch_uuid
check_row_count Port_Binding 1 logical_port=sw0-vir6 virtual_parent=sw0-p1
Expand Down Expand Up @@ -37001,7 +37001,7 @@ tpa=$(ip_to_hex 10 0 0 10)
send_garp 1 1 $eth_src $eth_dst $spa $tpa

OVS_WAIT_UNTIL([test 1 = `cat hv1/ovn-controller.log | grep "pinctrl received packet-in" | \
grep opcode=BIND_VPORT | grep OF_Table_ID=28 | wc -l`])
grep opcode=BIND_VPORT | grep OF_Table_ID=$(ovn-debug lflow-stage-to-oftable ls_in_arp_rsp) | wc -l`])

sleep_controller hv1

Expand Down

0 comments on commit de76bed

Please sign in to comment.