mbox series

[net-next,v2,00/12] selftests: mptcp: share code and fix shellcheck warnings

Message ID 20240306-upstream-net-next-20240304-selftests-mptcp-shared-code-shellcheck-v2-0-bc79e6e5e6a0@kernel.org
Headers show
Series selftests: mptcp: share code and fix shellcheck warnings | expand

Message

Matthieu Baerts (NGI0) March 6, 2024, 9:42 a.m. UTC
This series cleans MPTCP selftests code.

Patch 1 stops using 'iptables-legacy' if available, but uses 'iptables',
which is likely 'iptables-nft' behind.

Patches 2, 4 and 6 move duplicated code to mptcp_lib.sh. Patch 3 is a
preparation for patch 4, and patch 5 adds generic actions at the
creation and deletion of netns.

Patches 7 to 11 disable a few shellcheck warnings, and fix the rest, so
it is easy to spot real issues later. MPTCP CI is checking that now.

Patch 12 avoids redoing some actions at init time twice, e.g. restarting
the pm events tool.

Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
---
Changes in v2:
- The former patch 3/13 ("selftests: mptcp: add mptcp_lib_check_output
  helper") has been dropped, it was not supposed to be sent.
- Link to v1: https://lore.kernel.org/r/20240305-upstream-net-next-20240304-selftests-mptcp-shared-code-shellcheck-v1-0-66618ea5504e@kernel.org

---
Geliang Tang (5):
      selftests: mptcp: add mptcp_lib_check_tools helper
      selftests: mptcp: add local variables rndh
      selftests: mptcp: add mptcp_lib_ns_init/exit helpers
      selftests: mptcp: more operations in ns_init/exit
      selftests: mptcp: add mptcp_lib_events helper

Matthieu Baerts (NGI0) (7):
      selftests: mptcp: stop forcing iptables-legacy
      selftests: mptcp: diag: fix shellcheck warnings
      selftests: mptcp: connect: fix shellcheck warnings
      selftests: mptcp: sockopt: fix shellcheck warnings
      selftests: mptcp: pm netlink: fix shellcheck warnings
      selftests: mptcp: simult flows: fix shellcheck warnings
      selftests: userspace pm: avoid relaunching pm events

 tools/testing/selftests/net/mptcp/diag.sh          |  35 +++----
 tools/testing/selftests/net/mptcp/mptcp_connect.sh | 111 +++++++++++----------
 tools/testing/selftests/net/mptcp/mptcp_join.sh    |  60 ++---------
 tools/testing/selftests/net/mptcp/mptcp_lib.sh     |  68 +++++++++++++
 tools/testing/selftests/net/mptcp/mptcp_sockopt.sh |  55 +++-------
 tools/testing/selftests/net/mptcp/pm_netlink.sh    |  31 +++---
 tools/testing/selftests/net/mptcp/simult_flows.sh  |  38 +++----
 tools/testing/selftests/net/mptcp/userspace_pm.sh  |  56 +++--------
 8 files changed, 210 insertions(+), 244 deletions(-)
---
base-commit: 09fcde54776180a76e99cae7f6d51b33c4a06525
change-id: 20240304-upstream-net-next-20240304-selftests-mptcp-shared-code-shellcheck-160fceb5ce44

Best regards,

Comments

patchwork-bot+netdevbpf@kernel.org March 8, 2024, 5:10 a.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 06 Mar 2024 10:42:49 +0100 you wrote:
> This series cleans MPTCP selftests code.
> 
> Patch 1 stops using 'iptables-legacy' if available, but uses 'iptables',
> which is likely 'iptables-nft' behind.
> 
> Patches 2, 4 and 6 move duplicated code to mptcp_lib.sh. Patch 3 is a
> preparation for patch 4, and patch 5 adds generic actions at the
> creation and deletion of netns.
> 
> [...]

Here is the summary with links:
  - [net-next,v2,01/12] selftests: mptcp: stop forcing iptables-legacy
    https://git.kernel.org/netdev/net-next/c/7c2eac649054
  - [net-next,v2,02/12] selftests: mptcp: add mptcp_lib_check_tools helper
    https://git.kernel.org/netdev/net-next/c/3fb8c33ef4b9
  - [net-next,v2,03/12] selftests: mptcp: add local variables rndh
    https://git.kernel.org/netdev/net-next/c/4214aac14e51
  - [net-next,v2,04/12] selftests: mptcp: add mptcp_lib_ns_init/exit helpers
    https://git.kernel.org/netdev/net-next/c/3a0f9bed3c28
  - [net-next,v2,05/12] selftests: mptcp: more operations in ns_init/exit
    https://git.kernel.org/netdev/net-next/c/df8d3ba55b4f
  - [net-next,v2,06/12] selftests: mptcp: add mptcp_lib_events helper
    https://git.kernel.org/netdev/net-next/c/35bc143a8514
  - [net-next,v2,07/12] selftests: mptcp: diag: fix shellcheck warnings
    https://git.kernel.org/netdev/net-next/c/97633aa74d93
  - [net-next,v2,08/12] selftests: mptcp: connect: fix shellcheck warnings
    https://git.kernel.org/netdev/net-next/c/e3aae1098f10
  - [net-next,v2,09/12] selftests: mptcp: sockopt: fix shellcheck warnings
    https://git.kernel.org/netdev/net-next/c/5751c291349d
  - [net-next,v2,10/12] selftests: mptcp: pm netlink: fix shellcheck warnings
    https://git.kernel.org/netdev/net-next/c/21781b42f2f3
  - [net-next,v2,11/12] selftests: mptcp: simult flows: fix shellcheck warnings
    https://git.kernel.org/netdev/net-next/c/2aebd3579d90
  - [net-next,v2,12/12] selftests: userspace pm: avoid relaunching pm events
    https://git.kernel.org/netdev/net-next/c/c66fb480a330

You are awesome, thank you!