Re: [PATCH mptcp-next] selftests: mptcp: fix a mp_fail test warning

[Matthieu Baerts <matthieu.baerts@tessares.net>]

Hi Geliang,

On 03/05/2022 02:19, Geliang Tang wrote:
> Older version tc command (iprout2 5.3) shows 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.

Thank you for the patch!

Now in our tree (fix for net-next) with my RvB tag and without the
iproute2 typo:

New patches for t/upstream:
- d691758ed81c: selftests: mptcp: fix a mp_fail test warning
- Results: c3f5d94e8dc4..9cac72edfb62 (export)

Builds and tests are now in progress:

https://cirrus-ci.com/github/multipath-tcp/mptcp_net-next/export/20220503T163309
https://github.com/multipath-tcp/mptcp_net-next/actions/workflows/build-validation.yml?query=branch:export

Cheers,
Matt
-- 
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net