[Fuego] [PATCH 05/18] LTP: we need root permissions

Bird, Timothy Tim.Bird at sony.com
Wed Jan 31 04:53:30 UTC 2018


I think some LTP tests don't need root.  This is rather a big hammer,
and means that none of LTP will run, if the Fuego test user is
not root.

Do we know which ones will fail?  The LTP style guide
https://github.com/linux-test-project/ltp/wiki/StyleGuide
says for individual LTP tests to use tst_require_root()
if they need it, and to fall back to other behavior if
it's not available.

Please confirm that this global dependency check what you want
for LTP.
 -- Tim



> -----Original Message-----
> From: Daniel Sangorrin
> Signed-off-by: Daniel Sangorrin <daniel.sangorrin at toshiba.co.jp>
> ---
>  engine/tests/Functional.LTP/fuego_test.sh | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/engine/tests/Functional.LTP/fuego_test.sh
> b/engine/tests/Functional.LTP/fuego_test.sh
> index 2473cf1..1305421 100755
> --- a/engine/tests/Functional.LTP/fuego_test.sh
> +++ b/engine/tests/Functional.LTP/fuego_test.sh
> @@ -1,6 +1,8 @@
>  # FIXTHIS: use https://github.com/linux-test-project/ltp.git instead of a
> tarball
>  tarball=ltp-7eb77fbfd80.tar.bz2
> 
> +NEED_ROOT=1
> +
>  ALLTESTS="
>  admin_tools         fs_perms_simple       ltp-aiodio.part3  net_stress.appl
> quickhit
>  can                 fs_readonly           ltp-aiodio.part4  net_stress.broken_ip
> rpc_tests
> --
> 2.7.4
> 
> 
> _______________________________________________
> Fuego mailing list
> Fuego at lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego


More information about the Fuego mailing list