mbox series

[net,0/4] selftests: forwarding: Various fixes

Message ID 20240208155529.1199729-1-idosch@nvidia.com (mailing list archive)
Headers show
Series selftests: forwarding: Various fixes | expand

Message

Ido Schimmel Feb. 8, 2024, 3:55 p.m. UTC 
Fix various problems in the forwarding selftests so that they will pass
in the netdev CI instead of being ignored. See commit messages for
details.

Ido Schimmel (4):
  selftests: forwarding: Fix layer 2 miss test flakiness
  selftests: forwarding: Fix bridge MDB test flakiness
  selftests: forwarding: Suppress grep warnings
  selftests: forwarding: Fix bridge locked port test flakiness

 .../selftests/net/forwarding/bridge_locked_port.sh |  4 ++--
 .../testing/selftests/net/forwarding/bridge_mdb.sh | 14 +++++++++-----
 .../selftests/net/forwarding/tc_flower_l2_miss.sh  |  8 ++++++--
 3 files changed, 17 insertions(+), 9 deletions(-)

Comments

Hangbin Liu Feb. 9, 2024, 8:56 a.m. UTC | #1 
On Thu, Feb 08, 2024 at 05:55:25PM +0200, Ido Schimmel wrote:
> Fix various problems in the forwarding selftests so that they will pass
> in the netdev CI instead of being ignored. See commit messages for
> details.
> 
> Ido Schimmel (4):
>   selftests: forwarding: Fix layer 2 miss test flakiness
>   selftests: forwarding: Fix bridge MDB test flakiness
>   selftests: forwarding: Suppress grep warnings
>   selftests: forwarding: Fix bridge locked port test flakiness
> 
>  .../selftests/net/forwarding/bridge_locked_port.sh |  4 ++--
>  .../testing/selftests/net/forwarding/bridge_mdb.sh | 14 +++++++++-----
>  .../selftests/net/forwarding/tc_flower_l2_miss.sh  |  8 ++++++--
>  3 files changed, 17 insertions(+), 9 deletions(-)
> 
> -- 
> 2.43.0
> 

Reviewed-by: Hangbin Liu <liuhangbin@gmail.com>
Nikolay Aleksandrov Feb. 9, 2024, 10:39 a.m. UTC | #2 
On 2/8/24 17:55, Ido Schimmel wrote:
> Fix various problems in the forwarding selftests so that they will pass
> in the netdev CI instead of being ignored. See commit messages for
> details.
> 
> Ido Schimmel (4):
>    selftests: forwarding: Fix layer 2 miss test flakiness
>    selftests: forwarding: Fix bridge MDB test flakiness
>    selftests: forwarding: Suppress grep warnings
>    selftests: forwarding: Fix bridge locked port test flakiness
> 
>   .../selftests/net/forwarding/bridge_locked_port.sh |  4 ++--
>   .../testing/selftests/net/forwarding/bridge_mdb.sh | 14 +++++++++-----
>   .../selftests/net/forwarding/tc_flower_l2_miss.sh  |  8 ++++++--
>   3 files changed, 17 insertions(+), 9 deletions(-)
> 

Nice, for the series:
Acked-by: Nikolay Aleksandrov <razor@blackwall.org>
patchwork-bot+netdevbpf@kernel.org Feb. 9, 2024, 7:40 p.m. UTC | #3 
Hello:

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

On Thu, 8 Feb 2024 17:55:25 +0200 you wrote:
> Fix various problems in the forwarding selftests so that they will pass
> in the netdev CI instead of being ignored. See commit messages for
> details.
> 
> Ido Schimmel (4):
>   selftests: forwarding: Fix layer 2 miss test flakiness
>   selftests: forwarding: Fix bridge MDB test flakiness
>   selftests: forwarding: Suppress grep warnings
>   selftests: forwarding: Fix bridge locked port test flakiness
> 
> [...]

Here is the summary with links:
  - [net,1/4] selftests: forwarding: Fix layer 2 miss test flakiness
    https://git.kernel.org/netdev/net/c/93590849a05e
  - [net,2/4] selftests: forwarding: Fix bridge MDB test flakiness
    https://git.kernel.org/netdev/net/c/7399e2ce4d42
  - [net,3/4] selftests: forwarding: Suppress grep warnings
    https://git.kernel.org/netdev/net/c/dd6b34589441
  - [net,4/4] selftests: forwarding: Fix bridge locked port test flakiness
    https://git.kernel.org/netdev/net/c/f97f1fcc9690

You are awesome, thank you!