selftests: mptcp: fix a mp_fail test warning
authorGeliang Tang <geliang.tang@suse.com>
Sat, 14 May 2022 00:21:13 +0000 (17:21 -0700)
committerJakub Kicinski <kuba@kernel.org>
Mon, 16 May 2022 20:11:30 +0000 (13:11 -0700)
Old tc versions (iproute2 5.3) show actions in multiple lines, not a
single line. Then the following unexpected MP_FAIL selftest output
occurs:

 file received by server has inverted byte at 169
 ./mptcp_join.sh: line 1277: [: [{"total acts":1},{"actions":[{"order":0 pedit ,"control_action":{"type":"pipe"}keys 1
         index 1 ref 1 bind 1,"installed":0,"last_used":0
         key #0  at 148: val ff000000 mask ffffffff
 5: integer expression expected
 001 Infinite map                      syn[ ok ] - synack[ ok ] - ack[ ok ]
                                       sum[ ok ] - csum  [ ok ]
                                       ftx[ ok ] - failrx[ ok ]
                                       rtx[ ok ] - rstrx [ ok ]
                                       itx[ ok ] - infirx[ ok ]
                                       ftx[ ok ] - failrx[ ok ] invert

This patch adds a 'grep' before 'sed' to fix this.

Fixes: b6e074e171bc ("selftests: mptcp: add infinite map testcase")
Reviewed-by: Matthieu Baerts <matthieu.baerts@tessares.net>
Signed-off-by: Geliang Tang <geliang.tang@suse.com>
Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/mptcp/mptcp_join.sh

index d1de1e7..7381d1f 100755 (executable)
@@ -2693,6 +2693,7 @@ fastclose_tests()
 pedit_action_pkts()
 {
        tc -n $ns2 -j -s action show action pedit index 100 | \
+               grep "packets" | \
                sed 's/.*"packets":\([0-9]\+\),.*/\1/'
 }