Re: [LTP] [RFC PATCH 1/1] fs: Remove acl/tacl_xattr.sh

[Petr Vorel <pvorel@suse.cz>]

> Hi!
> > Old script which is not using LTP API, not in any runtest file.

> > It tests acl (getfacl, setfacl) and attr (getfattr, setfattr) tools, but
> > both project have their own tests [1] [2]. LTP concentrates on kernel
> > part of testing nowadays, therefore remove it.  Script is very simple
> > that it does not have any benefit for upstream.

> > [1] https://cgit.git.savannah.gnu.org/cgit/acl.git/tree/test
> > [2] https://cgit.git.savannah.gnu.org/cgit/attr.git/tree/test

> While I agree that this is testing an userspace tools we do not have
> much acl coverate in LTP, there seems to be some network nfsv4 tests but
> that seems to be all. I guess that we should at least create an github
> issue that we should write a few tests for that. I guess that writing a
> simple test that sets up right extended attributes and expecte a file to
> be accessible or not shouldn't be that hard.

Add test for filesystem extended attributes
https://github.com/linux-test-project/ltp/issues/1265

Please have a look to the ticket and improve it if needed.

I'm ok to keep this until we have a proper ACL test.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp