| Message ID | 20230310104729.32078-1-pvorel@suse.cz (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [ima-evm-utils] fsverity.test: Add /usr/sbin into $PATH | expand |
On Fri, 2023-03-10 at 11:47 +0100, Petr Vorel wrote: > Unlike GitHub CI, tests which run by packaging tools often run as non-root, > thus required tools aren't visible: > > ./functions.sh: line 45: type: mkfs: not found > mkfs is required for test > ./functions.sh: line 45: type: blkid: not found > blkid is required for test > ./functions.sh: line 45: type: e2fsck: not found > e2fsck is required for test > ./functions.sh: line 45: type: tune2fs: not found > tune2fs is required for test > > Signed-off-by: Petr Vorel <pvorel@suse.cz> > --- > Hi Mimi, > > some would maybe appreciate option to skip all tests, > but this is enough for me. Hi Petr, Thank you for the patch. It's now queued in next-testing. The UML kernel opens up a lot of options going forward. For now all the tests are lumped together, but they should be separated based on ima-evm-utils selftests vs. the kernel tests. Even that might not be granular enough. For now from the command line, the environment variable "TESTS" overrides the set of tests as defined in the Makefile to execute. For example, "export TESTS="ima_sign.test sign_verify.test"; make -e check" executes just the two tests. Also commit b573b7d4a186 ("Introduce TST_LIST variable to select a test to execute") can define the set of tests to execute.
Hi Mimi, > On Fri, 2023-03-10 at 11:47 +0100, Petr Vorel wrote: > > Unlike GitHub CI, tests which run by packaging tools often run as non-root, > > thus required tools aren't visible: > > ./functions.sh: line 45: type: mkfs: not found > > mkfs is required for test > > ./functions.sh: line 45: type: blkid: not found > > blkid is required for test > > ./functions.sh: line 45: type: e2fsck: not found > > e2fsck is required for test > > ./functions.sh: line 45: type: tune2fs: not found > > tune2fs is required for test > > Signed-off-by: Petr Vorel <pvorel@suse.cz> > > --- > > Hi Mimi, > > some would maybe appreciate option to skip all tests, > > but this is enough for me. > Hi Petr, > Thank you for the patch. It's now queued in next-testing. Thank you. > The UML kernel opens up a lot of options going forward. For now all > the tests are lumped together, but they should be separated based on > ima-evm-utils selftests vs. the kernel tests. Even that might not be > granular enough. Thanks for info, sounds reasonable. > For now from the command line, the environment variable "TESTS" > overrides the set of tests as defined in the Makefile to execute. For > example, "export TESTS="ima_sign.test sign_verify.test"; make -e check" > executes just the two tests. Also, TESTS= would skip all the tests (but prefer to run them). > Also commit b573b7d4a186 ("Introduce TST_LIST variable to select a test > to execute") can define the set of tests to execute. Good to know (this one is hidden in test functions). Kind regards, Petr
diff --git a/tests/fsverity.test b/tests/fsverity.test index 01d5c35..a274158 100755 --- a/tests/fsverity.test +++ b/tests/fsverity.test @@ -30,7 +30,7 @@ # custom policy rules might take precedence. cd "$(dirname "$0")" || exit 1 -PATH=../src:../fsverity-utils:$PATH +PATH=../src:../fsverity-utils:/usr/sbin:$PATH source ./functions.sh # Base VERBOSE on the environment variable, if set.
Unlike GitHub CI, tests which run by packaging tools often run as non-root, thus required tools aren't visible: ./functions.sh: line 45: type: mkfs: not found mkfs is required for test ./functions.sh: line 45: type: blkid: not found blkid is required for test ./functions.sh: line 45: type: e2fsck: not found e2fsck is required for test ./functions.sh: line 45: type: tune2fs: not found tune2fs is required for test Signed-off-by: Petr Vorel <pvorel@suse.cz> --- Hi Mimi, some would maybe appreciate option to skip all tests, but this is enough for me. Kind regards, Petr tests/fsverity.test | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)