Page MenuHomeFreeBSD

Enable build/install DTrace tests on amd64
AcceptedPublic

Authored by lwhsu on Apr 29 2019, 3:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 12:07 AM
Unknown Object (File)
Jul 11 2023, 10:40 PM
Unknown Object (File)
May 30 2023, 3:06 AM
Unknown Object (File)
Mar 21 2023, 8:21 PM
Unknown Object (File)
Mar 4 2023, 12:16 AM
Unknown Object (File)
Jan 22 2023, 6:45 PM
Unknown Object (File)
Dec 20 2022, 6:44 AM
Unknown Object (File)
Nov 30 2022, 5:27 AM

Details

Reviewers
markj
emaste
gnn
Summary

Enable build/install DTrace tests on amd64.

Also need to regen src.conf.5

Test Plan

cd /usr/tests/cddl/usr.sbin/dtrace && kyua test

Also, https://ci.freebsd.org/job/FreeBSD-head-amd64-dtrace_test/ is green now.
Flaky tests, if found, should be mark skip.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 23969
Build 22877: arc lint + arc unit

Event Timeline

Looks fine to me, but I think we need to fix the flaky tests first.

Looks fine to me, but I think we need to fix the flaky tests first.

Rather, if the intention is to skip flaky tests, we should let the DTrace job for some time to ensure that we can identify them all. I know of one, for which you submitted a PR, but I believe there are others.

Looks fine to me, but I think we need to fix the flaky tests first.

Rather, if the intention is to skip flaky tests, we should let the DTrace job for some time to ensure that we can identify them all. I know of one, for which you submitted a PR, but I believe there are others.

Agree, I am not intended to commit it immediately. But I think it's time to start to discuss turning this on by default. :-)

Agree, I am not intended to commit it immediately. But I think it's time to start to discuss turning this on by default. :-)

Oh, and we don't have to wait for too long, there are some other flaky test cases in head so we can also deal with them together.

share/mk/src.opts.mk
408

Do we know how the tests do on other archs? I think it's fine to do this short-term but we should plan to have the tests on for all architectures where DTrace exists.

share/mk/src.opts.mk
408

It'll take substantial effort to get them all passing on i386. I'm not sure about other arches, but I expect the test suite is in pretty bad shape everywhere except amd64.

share/mk/src.opts.mk
408

Is it because the tests are inherently amd64-specific, or because DTrace does not work well on other architectures? Or both?

share/mk/src.opts.mk
408

Mostly the latter, though it's not really that it doesn't "work well" so much as there are missing arch-dependent pieces of the port.

This revision is now accepted and ready to land.May 7 2019, 3:32 PM