Created attachment 203225 [details] patch to require capsicum feature for a test On a system without capsicum, ioctls_test:cap_ioctls__listen_copy test failed and should be skipped: Before the patch: # kyua test sys/capsicum/ioctls_test sys/capsicum/ioctls_test:cap_ioctls__listen_copy -> failed: /var/jenkins/workspace/ocafirmware_build-master/FreeBSD/tests/sys/capsicum/io$ tls_test.c:94: cap_rights_limit(s[0], &rights) == 0 not met [0.003s] Results file id is usr_tests.20190328-212133-379353 Results saved to /root/.kyua/store/results.usr_tests.20190328-212133-379353.db 0/1 passed (1 failed) After the patch: # kyua test sys/capsicum/ioctls_test sys/capsicum/ioctls_test:cap_ioctls__listen_copy -> skipped: kernel feature (security_capability) not present [0.003s] Results file id is usr_tests.20190328-213129-974398 Results saved to /root/.kyua/store/results.usr_tests.20190328-213129-974398.db 1/1 passed (0 failed)
Check the spelling on that feature. I think it should be either "security_capabilities" or "security_capability_mode".
Created attachment 203227 [details] patch v2 to require capsicum feature for a test Ouch, good catch! I need to add tests to tests my tests ;-)
Approved.
A commit references this bug: Author: olivier Date: Fri Mar 29 08:43:22 UTC 2019 New revision: 345681 URL: https://svnweb.freebsd.org/changeset/base/345681 Log: Skip test if feature security_capabilities is not available PR: 236863 Approved by: asomers MFC after: 1 month Sponsored by: Netflix Changes: head/tests/sys/capsicum/Makefile head/tests/sys/capsicum/ioctls_test.c
Thanks for your advice.
Switched it back to "in progress" until all MFC done.
MFC expected soon?
Ping
A commit references this bug: Author: emaste Date: Wed Jan 15 21:05:43 UTC 2020 New revision: 356768 URL: https://svnweb.freebsd.org/changeset/base/356768 Log: MFC r345681: Skip test if feature security_capabilities is not available PR: 236863 Sponsored by: The FreeBSD Foundation Changes: _U stable/12/ stable/12/tests/sys/capsicum/Makefile stable/12/tests/sys/capsicum/ioctls_test.c