Skip to content

Add dependency checking

Jan Stancek ltp 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 4 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 4 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [LTP] [PATCH 0/4] Add dependency checking

----- Original Message -----

Quoted 5 lines

Hi,

While investigating failures from nightly runs, I came across some tests that don't explicitly check for dependencies.

Quoted 8 lines

Add appropriate check for dependencies so the tests are reported as an issue with the configuration rather than a test failure. These set of patches transitions ~50-60 tests from reporting failures to a configuration issue.

Thanks,
Punit

Hi,

Pushed with the exception of patch 2.

Switching quota_remount_test01.sh away from apicmds will likely need more changes, e.g. it should use at least tst_exit().

If we take the effort to port this test to shell library, it should be "tst_test.h" one.

If you're up to rewriting test for tst_test.h, that would be preffered. Otherwise, I rather use some shell check and avoid including "test.h" for the sole purpose of having tst_check_cmds.

Regards,
Jan