| Message ID | 20230713-tc-selftests-lkft-v1-0-1eb4fd3a96e7@tessares.net (mailing list archive) |
|---|---|
| Headers | show |
| Series | selftests: tc: increase timeout and add missing kconfig | expand |
On Thu, Jul 13, 2023 at 5:17 PM Matthieu Baerts <matthieu.baerts@tessares.net> wrote: > > When looking for something else in LKFT reports [1], I noticed that the > TC selftest ended with a timeout error: > > not ok 1 selftests: tc-testing: tdc.sh # TIMEOUT 45 seconds > > I also noticed most of the tests were skipped because the "teardown > stage" did not complete successfully. It was due to missing kconfig. > > These patches fix these two errors plus an extra one because this > selftest reads info from "/proc/net/nf_conntrack". Thank you Pedro for > having helped me fixing these issues [2]. > > Link: https://qa-reports.linaro.org/lkft/linux-next-master/build/next-20230711/testrun/18267241/suite/kselftest-tc-testing/test/tc-testing_tdc_sh/log [1] > Link: https://lore.kernel.org/netdev/0e061d4a-9a23-9f58-3b35-d8919de332d7@tessares.net/T/ [2] > Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net> For the patchset: Acked-by: Jamal Hadi Salim <jhs@mojatatu.com> Thanks for all the effort Matthieu! cheers, jamal > --- > Matthieu Baerts (3): > selftests: tc: set timeout to 15 minutes > selftests: tc: add 'ct' action kconfig dep > selftests: tc: add ConnTrack procfs kconfig > > tools/testing/selftests/tc-testing/config | 2 ++ > tools/testing/selftests/tc-testing/settings | 1 + > 2 files changed, 3 insertions(+) > --- > base-commit: 9d23aac8a85f69239e585c8656c6fdb21be65695 > change-id: 20230713-tc-selftests-lkft-363e4590f105 > > Best regards, > -- > Matthieu Baerts <matthieu.baerts@tessares.net> >
Hi David, Jakub, Paolo, On 13/07/2023 23:16, Matthieu Baerts wrote: > When looking for something else in LKFT reports [1], I noticed that the > TC selftest ended with a timeout error: > > not ok 1 selftests: tc-testing: tdc.sh # TIMEOUT 45 seconds > > I also noticed most of the tests were skipped because the "teardown > stage" did not complete successfully. It was due to missing kconfig. > > These patches fix these two errors plus an extra one because this > selftest reads info from "/proc/net/nf_conntrack". Thank you Pedro for > having helped me fixing these issues [2]. It looks like this series is marked as "Changes Requested" on Patchwork [1] but I think that's a mistake. There was one discussion on-going on the first patch but it looks like the proposed version is OK. I didn't see any instructions to pw-bot and nothing on the website [2]. Do you prefer if I re-send it? Cheers, Matt [1] https://patchwork.kernel.org/project/netdevbpf/list/?series=765455&state=* [2] https://patchwork.hopto.org/pw-bot.html
Hello: This series was applied to netdev/net.git (main) by Jakub Kicinski <kuba@kernel.org>: On Thu, 13 Jul 2023 23:16:43 +0200 you wrote: > When looking for something else in LKFT reports [1], I noticed that the > TC selftest ended with a timeout error: > > not ok 1 selftests: tc-testing: tdc.sh # TIMEOUT 45 seconds > > I also noticed most of the tests were skipped because the "teardown > stage" did not complete successfully. It was due to missing kconfig. > > [...] Here is the summary with links: - [net,1/3] selftests: tc: set timeout to 15 minutes https://git.kernel.org/netdev/net/c/fda05798c22a - [net,2/3] selftests: tc: add 'ct' action kconfig dep https://git.kernel.org/netdev/net/c/719b4774a8cb - [net,3/3] selftests: tc: add ConnTrack procfs kconfig https://git.kernel.org/netdev/net/c/031c99e71fed You are awesome, thank you!
When looking for something else in LKFT reports [1], I noticed that the TC selftest ended with a timeout error: not ok 1 selftests: tc-testing: tdc.sh # TIMEOUT 45 seconds I also noticed most of the tests were skipped because the "teardown stage" did not complete successfully. It was due to missing kconfig. These patches fix these two errors plus an extra one because this selftest reads info from "/proc/net/nf_conntrack". Thank you Pedro for having helped me fixing these issues [2]. Link: https://qa-reports.linaro.org/lkft/linux-next-master/build/next-20230711/testrun/18267241/suite/kselftest-tc-testing/test/tc-testing_tdc_sh/log [1] Link: https://lore.kernel.org/netdev/0e061d4a-9a23-9f58-3b35-d8919de332d7@tessares.net/T/ [2] Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net> --- Matthieu Baerts (3): selftests: tc: set timeout to 15 minutes selftests: tc: add 'ct' action kconfig dep selftests: tc: add ConnTrack procfs kconfig tools/testing/selftests/tc-testing/config | 2 ++ tools/testing/selftests/tc-testing/settings | 1 + 2 files changed, 3 insertions(+) --- base-commit: 9d23aac8a85f69239e585c8656c6fdb21be65695 change-id: 20230713-tc-selftests-lkft-363e4590f105 Best regards,